If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

jsonrpc API requests do not work when "Require API-Key authentication for API requests" is enabled due to missing API tokens

RESOLVED FIXED

Status

()

bugzilla.mozilla.org
Extensions: MyDashboard
--
major
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: dkl, Assigned: dkl)

Tracking

Production

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

2 years ago
SSIA
we call jsonrpc in a few more locations - 

js/comment-tagging.js
js/bug.js
js/instant-search.js
js/field.js
extensions/GuidedBugEntry/web/js/guided.js
extensions/MyDashboard/web/js/flags.js
extensions/MyDashboard/web/js/query.js
extensions/EditComments/web/js/editcomments.js
extensions/Review/web/js/review_history.js
extensions/REMO/template/en/default/pages/remo-form-payment.html.tmpl
Summary: my dashboard API requests do not work when 2fa is enabled due to missing API tokens → jsonrpc API requests do not work when "Require API-Key authentication for API requests" is enabled due to missing API tokens
(Assignee)

Comment 2

2 years ago
Created attachment 8655461 [details] [diff] [review]
1200610_1.patch
Attachment #8655461 - Flags: review?(glob)
(Assignee)

Comment 3

2 years ago
Created attachment 8655471 [details] [diff] [review]
1200610_2.patch

Found two other places where we need api_token.
Attachment #8655461 - Attachment is obsolete: true
Attachment #8655461 - Flags: review?(glob)
Attachment #8655471 - Flags: review?(glob)
Comment on attachment 8655471 [details] [diff] [review]
1200610_2.patch

Review of attachment 8655471 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob
Attachment #8655471 - Flags: review?(glob) → review+
(Assignee)

Comment 5

2 years ago
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   c5c9d67..7c39ca2  master -> master
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.