abstrask commented on PR #9231: URL: https://github.com/apache/devlake/pull/9231#issuecomment-6213586282
If you'd prefer a smaller UI-only change instead, following the approach from #9159, this would also fix the trigger case: ```diff +// Blueprint triggers can return 401 from a remote (e.g. GitHub App token request), not a DevLake session expiry. +const isBlueprintTrigger = (url?: string) => !!url && /\/blueprints\/[^/]+\/trigger\/?$/.test(url); @@ - !isPluginConnectionTest(requestUrl) + !isPluginConnectionTest(requestUrl) && + !isBlueprintTrigger(requestUrl) ``` It only covers that one endpoint, though; other endpoints surfacing an upstream 401 would still redirect to `/login`. Happy to switch to this if you prefer. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
