Skip to content

Comments

armbianmonitor: call curl with --fail flag so server 400/500 errors actually trigger a retry#7487

Merged
rpardini merged 1 commit intoarmbian:mainfrom
rpardini:pr/armbianmonitor-call-curl-with---fail-flag-so-server-400500-errors-actually-trigger-a-retry
Nov 17, 2024
Merged

armbianmonitor: call curl with --fail flag so server 400/500 errors actually trigger a retry#7487
rpardini merged 1 commit intoarmbian:mainfrom
rpardini:pr/armbianmonitor-call-curl-with---fail-flag-so-server-400500-errors-actually-trigger-a-retry

Conversation

@rpardini
Copy link
Member

armbianmonitor: call curl with --fail flag so server 400/500 errors actually trigger a retry

  • armbianmonitor: call curl with --fail flag so server 400/500 errors actually trigger a retry
    • otherwise, curl thinks a 4xx or 5xx error is perfectly acceptable. who knew?

…rs actually trigger a retry

- otherwise, curl thinks a 4xx or 5xx error is perfectly acceptable. who knew?
@rpardini rpardini requested a review from EvilOlaf November 17, 2024 12:57
@github-actions github-actions bot added the size/small PR with less then 50 lines label Nov 17, 2024
@rpardini rpardini marked this pull request as ready for review November 17, 2024 12:57
@rpardini rpardini requested a review from a team as a code owner November 17, 2024 12:57
@github-actions github-actions bot added Needs review Seeking for review BSP Board Support Packages labels Nov 17, 2024
@rpardini rpardini merged commit d66285d into armbian:main Nov 17, 2024
@EvilOlaf
Copy link
Member

EvilOlaf commented Nov 25, 2024

There seems yet another issue with the paste server. This is actually the 2nd report of this error: https://forum.armbian.com/topic/47374-orangepi-4-lts-spidev/#findComment-206729
Not sure if this has anything to do with curl, I actually doubt, but not sure what else could be...
@TheLinuxBug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BSP Board Support Packages Needs review Seeking for review size/small PR with less then 50 lines

Development

Successfully merging this pull request may close these issues.

2 participants