Sitelet https://github.com/simplecrawler/simplecrawler/pull/3
Skip to content
This repository was archived by the owner on Mar 7, 2021. It is now read-only.

Updating discoverResources function to ensure that matched href or src attributes always have a space in front of them. - #3

Merged
cgiffard merged 1 commit into
simplecrawler:masterfrom
mmoulton:master
Jul 19, 2012
Merged

cgiffard merged 1 commit into
simplecrawler:masterfrom
mmoulton:master

Conversation

@mmoulton

Copy link
Copy Markdown
Contributor

... This resolves an issue we have with incorrect matches on embedded GoogleAnalytics code.

The standard GA code has something like:

ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';

This was causing request for resources that did not exist.

…em. This resolves an issue we have with incorrect matches on embedded GoogleAnalytics code.
@cgiffard

Copy link
Copy Markdown
Member

Thanks! Looks good. :)

cgiffard added a commit that referenced this pull request Jul 19, 2012
Updating discoverResources function to ensure that matched `href` or `src` attributes always have a space in front of them.
@cgiffard
cgiffard merged commit c857d58 into simplecrawler:master Jul 19, 2012
@mmoulton

Copy link
Copy Markdown
Contributor Author

Great, thanks. Any idea when you are planning to perform your next release?

@cgiffard

Copy link
Copy Markdown
Member

There's a few things I'm working on - a better mongodb cache mechanism and some general tidying up.

Did you have anything specific you were after?

@cgiffard

Copy link
Copy Markdown
Member

Or were you referring to NPM? I can push an update now. :)

@mmoulton

Copy link
Copy Markdown
Contributor Author

I was just referring to NPM. If you can push soon, that would be great.

@cgiffard

Copy link
Copy Markdown
Member

That's done now. 0.0.6 should be the latest version - let me know if everything works OK for you! :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants