| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
The only issue I see is that I could pass something as dumb as: url('httpx://crazy.example', validate_scheme=lambda x: True)This then resolves to: #396 (comment) |
Sorry, something went wrong.
I do not understand what is the issue mentioned above, and what part of the linked comment is relevant. Users want to have control over the accepted url schemes, because their problems define the schemes to work with which are not universal (e.g. only svn+ssh://... URLs makes sense in their problem domain). Now some potential users can not use the functionality in the library, because the scheme check is too strict for them. In the end, doing something dumb is the users responsibility - maybe what they end up with is a novel, but valid use :) |
Sorry, something went wrong.
|
@yozachar is there anything else I could do to get it merged and released? |
Sorry, something went wrong.
Enhances `validators.url` to allow
- restricting the allowed schemes (e.g. to accept only https,
and nothing else)
- relaxing the allowed schemes to also accept less known schemes
(e.g. ws, wss, ldap, ...)
|
Apologies for the delay, thank you for the PR. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Enhances validators.url to allow
This backward compatible change let's library users resolve the problematic cases requested/discussed in