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

Kevin Rushforth kcr at openjdk.java.net
Thu Jan 6 21:57:23 UTC 2022


On Mon, 3 Jan 2022 18:32:54 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> 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

Looks good.

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

Marked as reviewed by kcr (Reviewer).

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


More information about the skara-dev mailing list