RFR: 1295: CSRBot throws intermittent exception when looking up JiraIssue links [v2]

Erik Joelsson erikj at openjdk.java.net
Mon Jan 3 18:32:54 UTC 2022


> This patch adds another retry condition for rest requests. When a rest call uses authentication but fails with 401 "unauthorized", we need to give it at least one more try. The authentication mechanism for JBS works with time limited tokens that sometimes need to be renewed. This can only be detected through failing with 401. On the retry, the dynamic authorization provider (authGen.getAuthHeaders()) will get to run again and will detect the stale token.
> 
> I'm currently running with this patch live for the affected bot and so far it's looking good, but I will leave it running a while more to make sure.

Erik Joelsson has updated the pull request incrementally with two additional commits since the last revision:

 - Merge branch 'SKARA-1295-csr-exception' of https://github.com/erikj79/skara into SKARA-1295-csr-exception
 - Overwrite old auth headers on retry

-------------

Changes:
  - all: https://git.openjdk.java.net/skara/pull/1268/files
  - new: https://git.openjdk.java.net/skara/pull/1268/files/625e5254..fc65fe36

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=skara&pr=1268&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=skara&pr=1268&range=00-01

  Stats: 29 lines in 1 file changed: 22 ins; 3 del; 4 mod
  Patch: https://git.openjdk.java.net/skara/pull/1268.diff
  Fetch: git fetch https://git.openjdk.java.net/skara pull/1268/head:pull/1268

PR: https://git.openjdk.java.net/skara/pull/1268


More information about the skara-dev mailing list