Bug 63048

Hello,
Please provide a simple html page that embeds a css and the corresponding css fole where images are not extracted.

Created attachment 36352 [details]
Jmeter file

jmx file to replay the issue.

Created attachment 36353 [details]
home page image

shows the requests to home page.

Created attachment 36354 [details]
Replay in jmeter

shows the replay to request to /

Sorry, I DO requests filtering.
So, I trust the request will retrieve all embedded resources if the option in the sampler is marked.
However, the files that referred in css are not requested.
When I visit the home page of Drupal sandbox, https://dm9l7.ply.st, I see 7 requests. THe sandboxes last 24 hours, If it has expired, you have to create another one, CHoose Drupal Core and 8.6.2.
The jmeter jmx is attached in order to replay this isssue and you can see the images too. 
For example, enable developer tools and visit the home page and you will see 7 requests. Replay the same request to / in Jmeter and you will see just 5 requests.

"CHoose Drupal Core and 8.6.2."
The site that I referred in previous comments is simplytest.me

Hello Flavio,

Creating a sandbox is not easy for me.

Could you please:

- Do it again
- Using a fresh / clear cache browser instance, call the page and save as HAR
or save HTML + resources as a zip
- Show which requests are not being saved

Are you sure missing requests are triggered by CSS and not javascript executed code ?

Thank you

The missing images are referred in css files using background-image .
The missing images are loupe.svg and feed.svg

The procedure to create the sandbox is extremely simple.

Just go to simplytest.me and choose Drupal Core and version 8.6.2 and click launch sandbox button.

I've saved the har file.

Created attachment 36366 [details]
har file

Home navegation

(In reply to Flavio Pepino from comment #9)
> Created attachment 36366 [details]
> har file
> 
> Home navegation

Thank you Flavio.

So I analyzed , there are 2 additional URLs:

- https://dmdv0.ply.st/core/misc/favicon.ico 
- https://dmdv0.ply.st/core/misc/icons/505050/loupe.svg


The first one is favicon.ico, we could probably add this one
The second one is because by default we don't download url() in CSS file as their potential downloading depends on display parameters in browser, information that we don't have.

Author: pmouawad
Date: Mon Jan 14 22:30:00 2019
New Revision: 1851300

URL: http://svn.apache.org/viewvc?rev=1851300&view=rev
Log:
Bug 63048 - JMeter does not retrieve link resources of type "shortcut icon" or "icon"

Contributed by UbikLoadPack (https://ubikloadpack.com)
Bugzilla Id: 63048

Modified:
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/JsoupBasedHtmlParser.java
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/LagartoBasedHtmlParser.java
    jmeter/trunk/test/src/org/apache/jmeter/protocol/http/parser/TestBug60842HtmlParser.java
    jmeter/trunk/xdocs/changes.xml

Hello Flavio,
favicon.ico is now downloaded.

For the other resources, for now we won't be able to improve much.

Sorry for my ignorance about your work. Will we see the correction in Jmeter 5.1?

You can already try nightly build:

- https://ci.apache.org/projects/jmeter/nightlies/

And yes it will be in 5.1

Philippe
Thanks for you attention.
I will wait the official release.

(In reply to Flavio Pepino from comment #15)
> Philippe
> Thanks for you attention.
> I will wait the official release.

It would be nice if you could at least make a test with nightly build.

It would be annoying to discover an issue after release.
Thanks

I will check out the nightly build.

While trying the nightly build I saw the icon's problem is solved.

This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/4965