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 connector for copyparty #3616
Conversation
Hi @9001 thanks for the contribution! Unfortunately we just released v3 of the extension which went through a whole TypeScript rewrite. The code is pretty much the same except requires Also the Let me know if you need help |
Nice, congratulations on v3 🎉 can't wait to see the new settings UI! I have rebased the branch on master; just let me know if it would be preferable to scrap the branch entirely and start afresh :> |
Looks like something went wrong as there is still conflicts + lots of changes :/ Might be worth applying your changes fresh against the our master? |
Sorry about that!! Think I got it right this time :> |
* better np_url retrieval
* actually retrieve np_url
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution!
Describe the changes you made
This adds support for copyparty, a self-hosted file sharing service. As most copyparty instances are for private use, I've included patterns for the public demo server and for the default port when running the server locally.
These changes will work on all copyparty versions starting from v1.6.11 onwards. If you'd like to test the changes, please drop by the demo server :>
Additional context
The same metadata is available through the media session api; however, the album cover URL inside the mediasession tags has a login token included as a URL parameter. Since everything works as expected even without this token, it felt like a good idea to strip it away before passing the image URL to the extension.