Skip to content
This repository was archived by the owner on Jan 7, 2019. It is now read-only.

Conversation

cilim
Copy link
Contributor

@cilim cilim commented Sep 23, 2016

PR for #48

@coveralls
Copy link

coveralls commented Sep 23, 2016

Coverage Status

Coverage remained the same at 97.25% when pulling 28a6983 on mc-sphinx-documentation into 145a2da on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.25% when pulling 2f8a298 on mc-sphinx-documentation into 145a2da on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.25% when pulling 07b9425 on mc-sphinx-documentation into 145a2da on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.25% when pulling 68fc7a5 on mc-sphinx-documentation into 145a2da on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.25% when pulling aff5864 on mc-sphinx-documentation into 145a2da on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.509% when pulling e195a79 on mc-sphinx-documentation into e074f01 on master.

@cilim cilim added the review pr label Nov 7, 2016

If you are building a web application that serves traditional HTML pages, or a
Single Page Application (Angular/React), this library will handle the cookie
sessions for you. No special configuration is necessary.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cookies for you*

sessions for you. No special configuration is necessary.

To use cookie authentication, simply use the ``require_authentication!`` before callback:

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

before action*


Here is an example configuration block, with the default settings:

.. code-block:: ruby
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code-block should be yaml, not ruby.

Token Validation Strategy
.........................

When a request comes into your server, this gem will use the Access Token
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to your server*


Opt-in to ``stormpath`` validation with this configuration:

.. code-block:: ruby
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same thing with code-block. Please change it in all places needed.

authorization and security, this strategy should be used. In this situation
the developer does a one-time exchange of their API Keys for an Access Token.
This Access Token is time limited and must be periodically refreshed. This adds a
layer of security, at the cost of being less simple than HTTP Basic
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would use more complex instead of less simple :)

- Use Faker to generate random test data
- Rename all user instances to account

Released on Aug 22, 2016
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't a Version 2.0.1 be placed on top of this?

@coveralls
Copy link

coveralls commented Nov 8, 2016

Coverage Status

Coverage decreased (-31.4%) to 66.102% when pulling 3cb9452 on mc-sphinx-documentation into e074f01 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.609% when pulling d033464 on mc-sphinx-documentation into d1c9a69 on master.

@cilim
Copy link
Contributor Author

cilim commented Nov 8, 2016

Comments fixed @DamirSvrtan

@cilim cilim merged commit 8fa1192 into master Nov 8, 2016
@cilim cilim deleted the mc-sphinx-documentation branch November 8, 2016 15:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants