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

Features: security, cache, default content type #5

Merged
merged 4 commits into from
Feb 6, 2016
Merged

Features: security, cache, default content type #5

merged 4 commits into from
Feb 6, 2016

Conversation

vladislavbakan
Copy link
Contributor

I can propose my features in next cases:

  1. Security. Implemented authenticate_or_request_with_http_basic for access to swagger
  2. Cache. Implemented swaggard doc caching
  3. Content-type. Implemented default content type for request and response (setting in configuration)
    I'm open for your comments and suggestions)
    P.S. It's cool thing, which i was searching for a long time)

@vladislavbakan
Copy link
Contributor Author

Sorry for one PR and one branch)

@adrian-gomez
Copy link
Owner

Hi, thanks a lot for your contribution and your kind words.
Would you mind updating the doc so that people knows about the new cool features :D

@vladislavbakan
Copy link
Contributor Author

OK, give me a little time for that) I'll update PR

@vladislavbakan
Copy link
Contributor Author

@adrian-gomez could you please see the new commit with documentation for new features and make your comments)

@adrian-gomez
Copy link
Owner

Merged thanks a lot for your contribution!
I'll make a new release with this changes.

adrian-gomez added a commit that referenced this pull request Feb 6, 2016
…enttype

Features: security, cache, default content type
@adrian-gomez adrian-gomez merged commit 6da8fb4 into adrian-gomez:master Feb 6, 2016
@vladislavbakan
Copy link
Contributor Author

Thank you for your base) It's a really very useful gem

@vladislavbakan vladislavbakan deleted the feature_security_cache_contenttype branch February 6, 2016 21:22
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.

2 participants