Skip to content

Update to 5.1.2 #20

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

Merged
merged 2 commits into from
Oct 22, 2024
Merged

Update to 5.1.2 #20

merged 2 commits into from
Oct 22, 2024

Conversation

cabutlermit
Copy link
Contributor

@cabutlermit cabutlermit commented Oct 21, 2024

What does this PR do?

  • Upgrade the Matomo Docker container from version 5.1.1 to 5.1.2
  • Upgrade the EnvironmentVariables plugin from version 5.0.0 to 5.0.2

Helpful background context

This was requested via JSM by UXWS. See release notes for more details from Matomo.

How can a reviewer manually see the effects of these changes?

Log in to the web UI as Superuser and verify that the reported version is 5.1.2.

Screenshot from Dev1 instance after deploying the new container:
image

Includes new or updated dependencies?

YES

  • Includes an update to the EnvironmentVariables plugin.
  • Requires a database upgrade after deploying the new container!

What are the relevant tickets?

Developer

  • [n/a] All new ENV is documented in README (or there is none)
  • Stakeholder approval has been confirmed (or is not needed)

Code Reviewer

  • The commit message is clear and follows our guidelines
    (not just this pull request message)
  • There are appropriate tests covering any new functionality
  • The documentation has been updated or is unnecessary
  • The changes have been verified
  • New dependencies are appropriate or there were no changes

Why these changes are being introduced:
A new release (5.0.2) of the EnvironmentVariables plugin is now
available from the Matomo Plugins directory.

How this addresses that need:
* Add the unzipped EnvironmentVariables 5.0.2 to the plugins list.

Side effects of this change:
None.
Why these changes are being introduced:
UXWS requested the upgrade to 5.1.2 to address security concerns.

How this addresses that need:
* Update the Dockerfile to pull the new 5.1.2 version of the source
container

Side effects of this change:
None.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/INFRA-483
@cabutlermit cabutlermit requested review from dhrutibc and a team October 21, 2024 20:47
Copy link

@zotoMIT zotoMIT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link

@dhrutibc dhrutibc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dev looks good

@cabutlermit cabutlermit merged commit b9fe38f into main Oct 22, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

3 participants