Skip to content

Conversation

@cheshirsky
Copy link
Member

Basic github auth module is added.

Copy link
Member

Choose a reason for hiding this comment

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

Like

@robhrt7
Copy link
Member

robhrt7 commented Feb 5, 2015

Please remove white borders from assets/i/unknown.gif

core/auth.js Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Never require something in runtime, do it on app start.

Copy link
Member Author

Choose a reason for hiding this comment

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

ok

@robhrt7
Copy link
Member

robhrt7 commented Feb 5, 2015

I couldn't make it work locally, even when I changed app secret and callback URL to http://local.host:8080/auth/done, could you please provide some instructions?

And since this auth will always need aditional setup, we should turn in off by default.

@cheshirsky
Copy link
Member Author

Please remove white borders from assets/i/unknown.gif

Done.

I couldn't make it work locally, even when I changed app secret and callback URL to http://local.host:8080/auth/done, could you please provide some instructions?

Both http://127.0.0.1:8080 and http:/localhost:8080 work fine, they just should be identical for callback and homepage URLs. You don't need any specific paths like '/auth/done'.

Would you like me to write some tips or doc for it?

And since this auth will always need aditional setup, we should turn in off by default.

Done.

Copy link
Member

Choose a reason for hiding this comment

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

please use .source_* namespace

core/auth.js Outdated
Copy link
Member

Choose a reason for hiding this comment

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

please move requires to top, before module exports

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@robhrt7
Copy link
Member

robhrt7 commented Feb 9, 2015

Use demo key, I've created it for localhost:8080

Client ID
cf00a9e7ee5d9d6af36f
Client Secret
aebe08e0aa66f6911e4f54df81ce64c9d6e0003b

@robhrt7
Copy link
Member

robhrt7 commented Feb 9, 2015

Add short doc about auth in /docs section, and define that it's disabled by default right now. Then do a squash commit to 0.4.1, I don't have any more comments.

@cheshirsky
Copy link
Member Author

Ok, I`ll do it asap.

@cheshirsky
Copy link
Member Author

Add short doc about auth in /docs section, and define that it's disabled by default right now. Then do a squash commit to 0.4.1, I don't have any more comments.

Short doc is added. I`m going to extend it by usage examples soon.

Use demo key, I've created it for localhost:8080

done.

cheshirsky added a commit that referenced this pull request Feb 17, 2015
@cheshirsky cheshirsky merged commit 9bbcbea into 0.4.1 Feb 17, 2015
@robhrt7 robhrt7 deleted the mikhailov/auth branch February 24, 2015 21:40
@robhrt7 robhrt7 restored the mikhailov/auth branch February 24, 2015 21:41
@robhrt7 robhrt7 deleted the mikhailov/auth branch March 13, 2015 16:23
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.

3 participants