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

Jump to bottom

Feature/plugin request termination #2328

Merged
merged 16 commits into from Apr 4, 2017
Merged

Feature/plugin request termination #2328

merged 16 commits into from Apr 4, 2017

Conversation

Tieske
Copy link
Member

@Tieske Tieske commented Apr 4, 2017

replaces #2051

pauldaustin and others added 16 commits April 4, 2017 09:22
@pauldaustin @Tieske
feat(tools): HTTP Response 503 Service unavailable
04ba3ac 
Add HTTP response for 503 Service unavailable. This will be used for a
new api-unavailable plugin that will send a 503 error. See #1279.
@pauldaustin @Tieske
Code style changes
d81a7dd 
Added , to end of list
Changed to only log 500 errors
@pauldaustin @Tieske
Implemented tests and updated comments
5c02ab3
@pauldaustin @Tieske
feat(plugin): request-termination
0557c0a 
A new plug-in that allows a request to be terminated and a specified
HTTP status code and body returned.

This is useful to temporarily return a status page for a service. For
example if the service is unavailable due to scheduled maintenance.
@pauldaustin @Tieske
feat(tools): HTTP Response 503 Service unavailable
30c5656 
Add HTTP response for 503 Service unavailable. This will be used for a
new api-unavailable plugin that will send a 503 error. See #1279.
@pauldaustin @Tieske
Code style changes
30400b1 
Added , to end of list
Changed to only log 500 errors
@pauldaustin @Tieske
Implemented tests and updated comments
04aa92c
@pauldaustin @Tieske
Removed custom table definitions as the config is stored directly in
ded20e3 
kong's plugins table
@pauldaustin @Tieske
in-line default vakue for status_code
3150263
@pauldaustin @Tieske
removed links to request-termination lua files that were no longer
e6f3dd8 
required
@pauldaustin @Tieske
Changed the list of built in plugins to be one per line. This makes it
a5153fd 
easier to merge changes
@pauldaustin @Tieske
Implemented changes as requested in pull request
91fdd52
@pauldaustin @Tieske
Removed accidental commit of kong.conf
b6af0e1
@pauldaustin @Tieske
Renamed variable error to err
bf57176
@Tieske
rename plugin directory
b5d1bd0
@Tieske
chore(doc) update changelog
d3fbc4a
@Tieske Tieske merged commit 020de3e into master Apr 4, 2017
@Tieske Tieske deleted the feature/plugin-request-termination branch April 4, 2017 08:32
This was referenced Apr 4, 2017
Closed
Closed
@Tieske Tieske added this to the 0.10.2 milestone Apr 4, 2017
@Tieske Tieske mentioned this pull request Apr 14, 2017
Closed
@p0pr0ck5 p0pr0ck5 mentioned this pull request Apr 21, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.10.2
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Tieske @pauldaustin