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

Bump http-proxy-middleware from 1.0.5 to 2.0.7 #3550

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

tdonohue
Copy link
Member

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

  • Verify all automated tests pass
  • If desired, test Sitemap and Signposting URLs (as described above). I've tested both and they work well after this update

@tdonohue tdonohue added dependencies Pull requests that update a dependency file 1 APPROVAL pull request only requires a single approval to merge port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release labels Oct 23, 2024
@tdonohue tdonohue added this to the 9.0 milestone Oct 23, 2024
@alanorth alanorth removed the port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release label Oct 24, 2024
@kshepherd
Copy link
Member

Note, there are also some small changes to package-lock.json for unrelated dependencies, changing dev to devOptional. The npm package.json docs say:

dev, optional, devOptional: If the package is strictly part of the devDependencies tree, then dev will be true. If it is strictly part of the optionalDependencies tree, then optional will be set. If it is both a dev dependency and an optional dependency of a non-dev dependency, then devOptional will be set. (An optional dependency of a dev dependency will have both dev and optional set.)

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?

Copy link
Member

@kshepherd kshepherd left a 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 !

@tdonohue
Copy link
Member Author

Regarding the changes to package-lock.json mentioned above...those modifications were automated by npm itself when I upgraded this dependency by running npm install http-proxy-middleware@^2.0.7.

So, I think you are correct, @kshepherd that it's simply npm doing some optimization / cleanup of its own after this upgrade.

@kshepherd kshepherd merged commit b4231dc into DSpace:main Oct 24, 2024
13 checks passed
@dspace-bot
Copy link
Contributor

Backport failed for dspace-8_x, because it was unable to cherry-pick the commit(s).

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

@tdonohue tdonohue deleted the update_http-proxy-middleware branch October 24, 2024 15:59
@tdonohue
Copy link
Member Author

The failed automated backport to 8.x above is expected.

This PR was manually ported to both 8.x and 7.x:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge dependencies Pull requests that update a dependency file port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

4 participants