-
Notifications
You must be signed in to change notification settings - Fork 433
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
Bump http-proxy-middleware from 1.0.5 to 2.0.7 #3550
Conversation
Note, there are also some small changes to
Everything works in testing though, so perhaps the changes to the dependency tree (removing that other version of http-proxy-middleware) altered things so the dep is not strict anymore? |
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.
+1 tests well and looks good (i made a note about some other changes to package-lock.json but they don't seem to be a problem and probably just are a result of the dep tree being cleaner) thanks @tdonohue !
Regarding the changes to So, I think you are correct, @kshepherd that it's simply npm doing some optimization / cleanup of its own after this upgrade. |
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin dspace-8_x
git worktree add -d .worktree/backport-3550-to-dspace-8_x origin/dspace-8_x
cd .worktree/backport-3550-to-dspace-8_x
git switch --create backport-3550-to-dspace-8_x
git cherry-pick -x 693adc7cfc8467b33f4bf4614ee33a0095a1f773 |
The failed automated backport to 8.x above is expected. This PR was manually ported to both 8.x and 7.x: |
References
Description
Bump
http-proxy-middleware
from 1.0.5 to 2.0.7.Tested that v2.0.7 works well with DSpace.
In the DSpace UI, this
http-proxy-middleware
is ONLY used for loading Sitemaps (e.g.[dspace.ui.url]/sitemap_index.html
when generated) and Signposting (embedded in Item pages under[dspace.ui.url]/signposting/*
URLs)Instructions for Reviewers