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.

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

http: universal url match #27

Open
lonerapier opened this issue Nov 4, 2024 · 1 comment
Open

http: universal url match #27

lonerapier opened this issue Nov 4, 2024 · 1 comment
Labels
feature ✨ New feature or core functionality

Comments

@lonerapier
Copy link
Collaborator

Currently, http_parse_and_lock_start_line file uses exact string match to check request URL, but some authenticated URL contains user-specific parameters like:

  • /v1/users/{{user_id}}/playlists
  • /v1/me/top/tracks?limit={{limit}}

Investigate:

  • solution possible:
    • regex
    • any other??
  • whether this should be added as a separate template based on constraint count

I feel regex might be overkill for initial use case, although zk-regex is an already audited library that we should first look towards.

@devloper
Copy link
Contributor

devloper commented Nov 4, 2024

I think some reduced regex could be great here, i.e. it must be "length x, and characters must be one of set_of_characters"

@0xJepsen 0xJepsen added the feature ✨ New feature or core functionality label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature ✨ New feature or core functionality
Projects
None yet
Development

No branches or pull requests

3 participants