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

[security] polyfill.io included via outdated RubixML fork #881

Open
tgoeg opened this issue Jul 2, 2024 · 4 comments
Open

[security] polyfill.io included via outdated RubixML fork #881

tgoeg opened this issue Jul 2, 2024 · 4 comments

Comments

@tgoeg
Copy link

tgoeg commented Jul 2, 2024

Not posting this secretly as this is already known across the web.

This apps composer.json includes rubixML via https://github.com/nextcloud-deps/RubixML
That repo is behind the current master of RubixML which has this already fixed.

Still including it:
https://github.com/nextcloud-deps/RubixML/blob/master/mkdocs.yml

Already fixed:
RubixML/ML@fef1033

I don't know who's responsible for the nextcloud-deps fork, but please either use another, current source or (make people) update the fork.
Thanks!

@kesselb
Copy link

kesselb commented Jul 6, 2024

Thank you, that's a good finding.

Please report it via https://hackerone.com/nextcloud.

@tgoeg
Copy link
Author

tgoeg commented Jul 8, 2024

Done.

@tgoeg
Copy link
Author

tgoeg commented Jul 8, 2024

2024-07-08_161832_screenshot
Well, that wasn't all that successful :-)

I still think placing vulnerable code on production systems is not the best idea, even if the code is not in use.
I may not see the other report, there's probably a more promising fix there.

@si458
Copy link

si458 commented Sep 26, 2024

came here to say the same thing, our plesk server went nuts with alerts, because its now listing it as a virus
image
simply removing the line fixes the issue as explained above
but nextcloud do need to fix this for the suspicious_login plugin and do a new release for production

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

No branches or pull requests

3 participants