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

[feature] transformations plugin #137

Closed
wants to merge 17 commits into from

Conversation

subnetmarco
Copy link
Member

This is the request transformation plugin, work in progress. #117

@thibaultcha thibaultcha added the idea/new plugin [legacy] those issues belong to Kong Nation, since GitHub issues are reserved for bug reports. label Apr 19, 2015
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.82%) to 65.88% when pulling 88e1e96 on feature/transformations-plugin into 2212535 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.82%) to 65.88% when pulling 88e1e96 on feature/transformations-plugin into 2212535 on master.

@subnetmarco
Copy link
Member Author

@thibaultcha this is ready, not really happy with the schema though. To add the plugin, execute a request like:

curl -d "name=request_transformer&api_id=ID&value.add.querystring=paramname:value&value.remove.form=param1, param2" 127.0.0.1:8001/plugins_configurations/

The reason why the param names and values are in the name:value format, is because with the current schema design there is no other way around but we have to use a separator, which in this case is :.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.82%) to 65.88% when pulling 16f4f4f on feature/transformations-plugin into ab69baa on master.

@thibaultcha thibaultcha changed the title Feature/transformations plugin [feature] transformations plugin Apr 21, 2015
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 567b097 on feature/transformations-plugin into * on master*.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.61%) to 65.91% when pulling b246b6d on feature/transformations-plugin into 2e3814d on master.

@subnetmarco
Copy link
Member Author

Merged and squashed with 1388516

@subnetmarco subnetmarco deleted the feature/transformations-plugin branch April 22, 2015 04:26
hutchic added a commit that referenced this pull request Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea/new plugin [legacy] those issues belong to Kong Nation, since GitHub issues are reserved for bug reports.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants