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

Using nginx variables in the regular expression #5

Open
nonhex opened this issue Apr 19, 2013 · 2 comments
Open

Using nginx variables in the regular expression #5

nonhex opened this issue Apr 19, 2013 · 2 comments

Comments

@nonhex
Copy link

nonhex commented Apr 19, 2013

I'm using nginx for many projects, there is a possibility to add the use of nginx variables in the query, for example: replace_filter /$project/ /, - or: replace_filter "/$project/" /

@agentzh
Copy link
Member

agentzh commented Apr 19, 2013

Nginx variables are not supported (yet) in the pattern arguments. And to make this feature really useful, we'll have to implement an LRU-based compiled regex cache for it as well, because regex compilation is expensive.

Patches welcome :)

agentzh added a commit that referenced this issue Dec 1, 2021
* feature: first draft of implementation (TEST 1 ~ 45 in 01-sanity.t are now passing).

* more progress.

* no -O2.

* tests: 04-capturing.t now fully passes.

* tests: 07-multi.t now passes fully.

* enabled more optimizations.

Co-authored-by: Yichun Zhang (agentzh) <[email protected]>
@bmwant
Copy link

bmwant commented May 11, 2022

hey, is there any update on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants