Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add nordbayern bridge #1513

Merged
merged 1 commit into from
Apr 3, 2020
Merged

add nordbayern bridge #1513

merged 1 commit into from
Apr 3, 2020

Conversation

theScrabi
Copy link
Contributor

So finally I added the bridge for for nordbayer without the nasty memory bug.
Turns out you need to free the HTML DOM yourself by running $myDOM->delete(); in the end. GC does not do this for you. Since I needed to scrape multiple sites at one run it broke the max memory :/.
Now its fixed.

fix intending
@teromene teromene merged commit badb531 into RSS-Bridge:master Apr 3, 2020
@teromene
Copy link
Member

teromene commented Apr 3, 2020

Thank you for the PR !

@theScrabi
Copy link
Contributor Author

Nice thx :)

@theScrabi theScrabi deleted the nordbayern branch April 4, 2020 11:23
infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this pull request Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants