Merge branch 'v3' into permission-manager

canonical · Jan 8, 2025 · 0958238 · 0958238
2 parents 7022a6c + 4ee7344
commit 0958238
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/internal/jujuclient/applicationoffers.go b/internal/jujuclient/applicationoffers.go
@@ -209,6 +209,11 @@ func (c Connection) GetApplicationOfferConsumeDetails(ctx context.Context, user
 			OfferURLs:     []string{info.Offer.OfferURL},
 			BakeryVersion: v,
 		},
+		// We need to specify the user tag, because the juju controller returns a macaroon
+		// with a third party caveat instructing the discharger to verify that the stated
+		// user has `consume` access to the offer. If the user tag is left empty juju
+		// will assume the logged-in user and since jimm is dialing the controller as
+		// `admin` user that causes an issue.
 		UserTag: user.String(),
 	}