Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Adding cloud auth for sync #82

Merged
merged 2 commits into from
Aug 14, 2017

Conversation

wtrocki
Copy link
Member

@wtrocki wtrocki commented Aug 14, 2017

Motivation

Adding cloud auth for sync

Progress

  • Protect sync calls

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 66e02a3 on wtrocki:RAINCATCH-1147 into ** on feedhenry-raincatcher:master**.

@wtrocki
Copy link
Member Author

wtrocki commented Aug 14, 2017

@JameelB we may need to add here the same pattern for role as in #75

@coveralls
Copy link

coveralls commented Aug 14, 2017

Coverage Status

Changes Unknown when pulling 1fc2535 on wtrocki:RAINCATCH-1147 into ** on feedhenry-raincatcher:master**.

@wtrocki
Copy link
Member Author

wtrocki commented Aug 14, 2017

Most of the changes here were made by @JameelB.
It looks like protect works only for passport. There is generic problem with keycloak that will be resolved in separate issue. I have created separate ticket to cover this work.

Copy link
Member Author

@wtrocki wtrocki left a comment

Choose a reason for hiding this comment

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

Reviewed, Verified.

@wtrocki wtrocki merged commit 29b7c88 into feedhenry-raincatcher:master Aug 14, 2017
@wtrocki wtrocki deleted the RAINCATCH-1147 branch August 14, 2017 20:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants