-
Notifications
You must be signed in to change notification settings - Fork 46
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
bug: normalize id elements to remove potential wrap characters #748
Conversation
Remove potential "{}" wrapper from bsoids and collection ids. (See original issue for details) Test included. Try sending in a sync request with a bso_id wrapped and URLencoded. Closes #680
Do these changes depend on |
This is probably just an aspect of my local build. Right now, I can't build or test this PR without #747 without doing a good deal of work to fish out the various interdependencies. |
So, reworked things to urldecode the bso_ids. Granted, now part of me is wondering if we should filter on those to prevent potential exploits by including high values. |
Description:
Remove potential "{}" wrapper from bsoids and collection ids. (See original issue for details)
Blocks on #747
Testing:
Test included.
Try sending in a sync request with a bso_id wrapped and URLencoded.
Issues:
Closes #680