aaron-wolmutt commented on code in PR #58016:
URL: https://github.com/apache/airflow/pull/58016#discussion_r2550359229
##########
airflow-core/src/airflow/ui/src/queries/useTrigger.ts:
##########
@@ -63,6 +64,33 @@ export const useTrigger = ({ dagId, onSuccessConfirm }: {
dagId: string; onSucce
};
const onError = (_error: unknown) => {
+ // Check if error is a 403 (Forbidden) status
+ if (isHttpError(_error, 403)) {
+ toaster.create({
+ description: translate(
+ "triggerDag.toaster.error.forbidden.description",
+ "You don't have permission to trigger this DAG",
+ ),
+ title: translate("triggerDag.toaster.error.forbidden.title",
"Permission Denied"),
+ type: "error",
+ });
+ } else if (isHttpError(_error, 401)) {
+ toaster.create({
+ description: translate(
+ "triggerDag.toaster.error.unauthorized.description",
+ "Please log in to trigger DAGs",
+ ),
+ title: translate("triggerDag.toaster.error.unauthorized.title",
"Authentication Required"),
+ type: "error",
+ });
Review Comment:
Yeah, if the JWT was invalid it would just redirect the user back to the
sign in. I removed the bug.
--
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]
