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

support to run header_filter and body_filter phase for global rules. #1364

Merged
merged 4 commits into from
Apr 2, 2020

Conversation

Akayeshmantha
Copy link
Member

Issues resolved

Fix #1338

lua/apisix.lua Outdated Show resolved Hide resolved
@spacewander
Copy link
Member

@Akayeshmantha
Maybe we can refactor the way to run global rule in log/header filter/body filter phases to use the same function?

@Akayeshmantha
Copy link
Member Author

Sure @spacewander created a new issue #1369 since this ticket mentioned a different issue

@Akayeshmantha
Copy link
Member Author

@spacewander @membphis

lua/apisix.lua Outdated Show resolved Hide resolved
@Akayeshmantha
Copy link
Member Author

@membphis can we merge this if its ok?

lua/apisix.lua Outdated Show resolved Hide resolved
@Akayeshmantha
Copy link
Member Author

@membphis done changes

@Akayeshmantha
Copy link
Member Author

@membphis @moonming if things ok can we merge this so I can work on the other issue related to this 🙂

@membphis
Copy link
Member

@Akayeshmantha you need to rebase you PR after c76e7e3 merged

@Akayeshmantha
Copy link
Member Author

@membphis rebased

@moonming
Copy link
Member

moonming commented Apr 1, 2020

@spacewander please take a look

@membphis membphis merged commit b61db08 into apache:master Apr 2, 2020
SaberMaster pushed a commit to SaberMaster/incubator-apisix that referenced this pull request Jun 30, 2020
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

Successfully merging this pull request may close these issues.

feature: support to run header_filter and body_filter phase for global rules
4 participants