Skip to content

feat: refresh token used for logs#154

Merged
lucaspin merged 4 commits intomasterfrom
refresh-token
Jul 11, 2022
Merged

feat: refresh token used for logs#154
lucaspin merged 4 commits intomasterfrom
refresh-token

Conversation

@lucaspin
Copy link
Copy Markdown
Contributor

@lucaspin lucaspin commented Jul 8, 2022

The tokens generated for agents to push logs to the Semaphore API have a short duration. Currently, if a job takes longer than the duration of the token, the job will proceed as usual, but the logs will be incomplete.

In order to address this, the agent needs to refresh the token. So now, if the agent gets a 401 when trying to push logs to the Semaphore API, it will refresh the token and try again.

@lucaspin lucaspin requested a review from shiroyasha July 8, 2022 21:39
@lucaspin lucaspin merged commit f49dc8e into master Jul 11, 2022
@lucaspin lucaspin deleted the refresh-token branch July 11, 2022 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant