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.

Sign up for GitHub

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

Jump to bottom

Update publish ui and service #1

Open
patkyn opened this issue Oct 28, 2024 · 3 comments
Open

Update publish ui and service #1

patkyn opened this issue Oct 28, 2024 · 3 comments
Assignees
@patkyn

Comments

@patkyn
Copy link
Collaborator

patkyn commented Oct 28, 2024 •
edited
Loading

  1. Upgrade React to use latest version

  2. Resolve some security vulnerability updates highlighted by dependabot

  3. Current version uses CAS for authentication for both publish ui and publish service. Migrate to cognito for authentication as CAS test is not working as expected.

  4. Other minor fixes for eg: licences with https value in dataset emls
@patkyn patkyn self-assigned this Oct 28, 2024
patkyn added a commit that referenced this issue Oct 29, 2024
@patkyn
#1 - Upgrade packages from dependabot security alerts
abd43ed
patkyn added a commit that referenced this issue Oct 29, 2024
@patkyn
#1 - Handle decoded id token from cognito
b141cd4
patkyn added a commit that referenced this issue Oct 29, 2024
@patkyn
#1 -Upgrade react and vite and auth packages
b1dd14f
patkyn added a commit that referenced this issue Oct 29, 2024
@patkyn
#1 - Publish ui to handle cognito, use id_token.
1dfe165
@patkyn patkyn mentioned this issue Oct 29, 2024
Merged
patkyn added a commit that referenced this issue Oct 29, 2024
@patkyn
Feature/issue#1 (#2)
95707dc 
* #1 - Upgrade packages from dependabot security alerts

* #1 - Handle decoded id token from cognito

* #1 -Upgrade react and vite and auth packages

* #1 - Publish ui to handle cognito, use id_token.
@patkyn
Copy link
Collaborator Author

patkyn commented Nov 7, 2024

Further warnings from the pytest version 6.2.5
image
Fix: Update the pytest package to above 7.2 as recommended by https://vulners.com/github/GHSA-W596-4WVX-J9J6

patkyn added a commit that referenced this issue Nov 7, 2024
@patkyn
#1 - update boto3
950090c
patkyn added a commit that referenced this issue Nov 7, 2024
@patkyn
#1 - fix vulnerability issue with pytest package
8bd0e62
@patkyn
Copy link
Collaborator Author

patkyn commented Dec 11, 2024

Pending issue: temporary file upload doesn't register user id from authenticated cognito users.

@patkyn
Copy link
Collaborator Author

patkyn commented Dec 11, 2024

Dockerfile changes: a1d2ad4

patkyn added a commit that referenced this issue Dec 11, 2024
@patkyn
#1 - Role and user id fix once authenticated via cognito, Error messa…
b2a1277 
…ges to align with publish service, allow pub description to be editable, minor fix to events
patkyn added a commit that referenced this issue Dec 11, 2024
@patkyn
#1 - Minor fix warning IDE warning message
4938e8c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@patkyn patkyn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@patkyn