Skip to content

upgrade panda to latest; partially switching to s3 sdk v2#4737

Merged
andrew-nowak merged 2 commits into
mainfrom
an/panda-upgrade
May 21, 2026
Merged

upgrade panda to latest; partially switching to s3 sdk v2#4737
andrew-nowak merged 2 commits into
mainfrom
an/panda-upgrade

Conversation

@andrew-nowak
Copy link
Copy Markdown
Member

What does this change?

Upgrade pan-domain-auth to latest version, which requires partially switching to the v2 s3 sdk.

How should a reviewer test this change?

How can success be measured?

Who should look at this?

Tested? Documented?

  • locally by committer
  • locally by Guardian reviewer
  • on the Guardian's TEST environment
  • relevant documentation added or amended (if needed)

@andrew-nowak andrew-nowak added the maintenance Departmental tracking: maintenance work, not a fix or a feature label May 14, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 14, 2026

@andrew-nowak andrew-nowak marked this pull request as ready for review May 14, 2026 16:30
@andrew-nowak andrew-nowak requested a review from a team as a code owner May 14, 2026 16:30
@bryophyta
Copy link
Copy Markdown
Contributor

This looks good to me on a code level! When testing in CODE though I found that I wasn't sure how the /cookieMonster endpoint is expected to work. It doesn't seem to be working exactly how I would have thought, so perhaps we can take a look together at some point?

Copy link
Copy Markdown
Contributor

@bryophyta bryophyta left a comment

Choose a reason for hiding this comment

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

lgtm!

@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented May 21, 2026

Seen on auth, usage, metadata-editor, leases, cropper, collections, media-api, kahuna (merged by @andrew-nowak 9 minutes and 59 seconds ago) Please check your changes!

@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented May 21, 2026

Seen on image-loader, thrall (merged by @andrew-nowak 12 minutes and 9 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants