Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

URIBone needed #1219

Closed
phorward opened this issue Jul 18, 2024 · 1 comment · Fixed by #1254
Closed

URIBone needed #1219

phorward opened this issue Jul 18, 2024 · 1 comment · Fixed by #1254
Assignees
@ArneGudermann
Labels
feature New feature or request help wanted Extra attention is needed idea Priority: Low This issue can be considered with enough idle time.

Comments

@phorward
Copy link
Member

Similar to #1155, why isn't there an URIBone to specify valid URLs and URIs?

The requirements must be evaluated, here are some ideas:

  • Accepted protocol (http, https...)
  • Accept, or accepted ports?
  • Local path, may start or not start with /
  • Clean-up
    • accept or deny GET-parameters
    • domains disallow list

Relates to #1218, I already missed such a bone for several use-cases in the past.
@phorward phorward added feature New feature or request help wanted Extra attention is needed idea labels Jul 18, 2024
@phorward phorward mentioned this issue Aug 27, 2024
Merged
@sveneberth sveneberth added the Priority: Low This issue can be considered with enough idle time. label Aug 30, 2024
@ArneGudermann ArneGudermann mentioned this issue Sep 6, 2024
Merged
@phorward phorward linked a pull request Oct 10, 2024 that will close this issue
feat: UriBone #1254
Merged
phorward added a commit that referenced this issue Oct 15, 2024
@ArneGudermann @phorward @sveneberth
feat: UriBone (#1254)
0c1da24 
Solution for #1219

---------

Co-authored-by: Jan Max Meyer <[email protected]>
Co-authored-by: Sven Eberth <[email protected]>
@phorward
Copy link
Member Author

Fixed by #1254

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

@ArneGudermann ArneGudermann

Labels
feature New feature or request help wanted Extra attention is needed idea Priority: Low This issue can be considered with enough idle time.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: UriBone ArneGudermann/viur-core
3 participants
@sveneberth @phorward @ArneGudermann