search

mikitex70 / plantuml-markdown

PlantUML plugin for Python-Markdown
BSD 2-Clause "Simplified" License
192 stars 55 forks source link

new: usr: retry on server errors or rate limits when rendering remotely #79

Closed nejch closed 1 year ago

nejch commented 1 year ago

Sometimes plantuml or kroki servers are behind reverse proxies/gateways that have rate limits in place, or may experience intermittent connection issues. The extension might fail when many diagrams are rendered because it makes too many burst requests.

This adds a custom keep-alive session for making requests in _render_remote_uml_image to retry a few times and respect rate-limit headers (e.g. https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/429).

More context: https://urllib3.readthedocs.io/en/latest/reference/urllib3.util.html#urllib3.util.Retry https://stackoverflow.com/questions/23267409/how-to-implement-retry-mechanism-into-python-requests-library

@mikitex70 I haven't added a config option here and just enabled it by default, but if you think it's necessary I can add one.

mikitex70 commented 1 year ago

Thanks @nejch, nice work!

anb0s commented 1 year ago

@mikitex70 sorry to bother you again, we would love to see this feature released soon as we have issues with a server enabled this "rate limit" and we have broken rendered pictures.

We are investigating a solution to use another server without the "rate limit", but may be new plantuml-markdown is already planed.

mikitex70 commented 1 year ago

I'm at work now, I'll release it tonight.

mikitex70 commented 1 year ago

Hi @nejch, I've just released the 3.7.2 which includes your PR. Excuse me for the delay, I did not understand that it was an urgent problem for you.

nejch commented 1 year ago

Thanks a lot @mikitex70 for the quick merge and release, and no worries! I did not either, until this morning when our users started reporting more issues :sweat_smile: