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

check operator version and provide correct presigned url #136

Open
wants to merge 55 commits into
base: main
Choose a base branch
from

Conversation

lizk886
Copy link
Contributor

@lizk886 lizk886 commented Jul 23, 2024

solution 2: get upgraded info by comparing version info in headers. With unit tests.

need to deploy IABTechLab/uid2-shared#314 first ( update the Const.java)

successful build: https://github.com/IABTechLab/uid2-core/actions/runs/10288910692
successful run in integration:

@lizk886 lizk886 changed the title initial code check operator version and provide correct presigned url Jul 23, 2024
@lizk886 lizk886 marked this pull request as ready for review July 23, 2024 21:39
@lizk886 lizk886 changed the base branch from main to wzh-uid2-3572-s3encryption-keys-distribution-endpoint July 24, 2024 23:53
Base automatically changed from wzh-uid2-3572-s3encryption-keys-distribution-endpoint to main August 1, 2024 21:12
@cYKatherine cYKatherine force-pushed the wzh-uid2-3574-check-version-update-presignedurl branch 5 times, most recently from 88ca356 to 349b3a3 Compare August 7, 2024 02:09
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.

4 participants