2024_06: BUG: Resolve authorization failures

lzim / teampsd

Team PSD is using GitHub, R and RMarkdown as part of our free and open science workflow.

GNU General Public License v3.0

9 stars 23 forks source link

2024_06: BUG: Resolve authorization failures #3163

Closed jamesmrollins closed 2 months ago

jamesmrollins commented 4 months ago

Resolve the following authorization failure. The current auth token is not valid for the desired group. Use the authentication PATCH from the encyclopedia:

PATCH .../api/v3/va/va-psd-dev/authentication {"objectType":"user", "groupKey":""}
Store the auth token using local storage or a cookie.
David B. will try to arrange John (Forio DEV) to discuss the local storage of authorization token on Monday night.

Resolve Estimate: 5/27

jamesmrollins commented 3 months ago

5/25 Standup

Current list of 400 series errors below.
Authorization error caused by the code looking for a Team Data file that is not identified by the user at this stage of the workflow.
Continuing to work on resolving these issues:

lzim commented 3 months ago

wk5 2024_05 #systems_sme_workflow @lzim @dlkibbe

@jamesmrollins - Unclear how this impacts our prep this AM. Please advise #systems_sme_workflow on the card. Thanks!

FYI: @dlounsbu

jamesmrollins commented 3 months ago

@tejaspesquare

Persistent 404 Errors, please resolve

jamesmrollins commented 3 months ago

@tejaspesquare

Persistent 400 Errors, Please Resolve

jamesmrollins commented 3 months ago

FYI @lzim @dlkibbe @lijenn @dlounsbu @ljmoody

5/29 Demo Call

(SME Key Information) These errors are not having an impact on functionality.
We suspect these are issues with the API call to the backend, but it is unclear what assets the API is trying to return (they are not human readable API returns).
Issue is returned to Forio for investigation

lzim commented 3 months ago

@tejaspesquare & #systems_sme_workflow

I'm trying to run down whether these have been resolved as we need to move on to prepping our MTL 3.7 Blue consults in PROD. Please provide an update. Thanks!

FYI: @jamesmrollins @lijenn @ljmoody

tejaspesquare commented 3 months ago

Hello @lzim,

yes, the matter has been successfully resolved. It pertains to the specific user and concerns the user's profile picture. Each user must upload a profile picture using an Avatar link.

Before Avtar

After Avtar

FYI- @jamesmrollins , @ljmoody , @lijenn

ljmoody commented 3 months ago

Per @tejaspesquare's reply above, I tested today in TEST-slow by:

Taking a screenshot of inspector console messages with no avatar associated with my profile:
Adding an avatar profile picture (jpg):
Taking a screenshot of inspector console messages after avatar was updated:

Moving this card to "done". FYI: @jamesmrollins