Skip to content

BB2-5048: Update node - #133

Merged
JamesDemeryNava merged 2 commits into
masterfrom
jamesdemery/bb2-5048-update-node
Aug 7, 2026
Merged

BB2-5048: Update node#133
JamesDemeryNava merged 2 commits into
masterfrom
jamesdemery/bb2-5048-update-node

Conversation

@JamesDemeryNava

Copy link
Copy Markdown
Contributor

JIRA Ticket:
BB2-5048

What Does This PR Do?

Updates the node version in the server/Dockerfile to address security vulnerabilities

What Should Reviewers Watch For?

If you're reviewing this PR, please check for these things in particular:

Validation

  • Build the project with docker-compose up -d
  • exec into the server image, run node -v, confirm you see v26.7.0
  • Go through an auth flow at localhost:3000 and make sure that works

@sb-DarenDean

Copy link
Copy Markdown

G2G! Have another eng review and approve. Thanks.

@ryan-morosa ryan-morosa self-assigned this Aug 7, 2026

@ryan-morosa ryan-morosa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It works as described! Nice job! I think you'll have to rerun the pipeline maybe

@JamesDemeryNava

Copy link
Copy Markdown
Contributor Author

@sb-DarenDean the client Dockerfile has the same node version, going to update that here as well.

@ryan-morosa ryan-morosa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Everything still looks good to me!

@JamesDemeryNava
JamesDemeryNava merged commit 3ae8010 into master Aug 7, 2026
3 checks passed
@JamesDemeryNava
JamesDemeryNava deleted the jamesdemery/bb2-5048-update-node branch August 7, 2026 17:15
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