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

Remove Auth info from Code #1

Open
jschappet opened this issue Apr 19, 2018 · 2 comments
Open

Remove Auth info from Code #1

jschappet opened this issue Apr 19, 2018 · 2 comments
Assignees
Labels
needs investigating The matter is open

Comments

@jschappet
Copy link

in Exist.js code you have your client ID:

auth: function() {
window.location = 'https://exist.io/oauth2/authorize?response_type=code&client_id=########&redirect_uri=https%3A%2F%2Fexist.redeclipse.net%2F&scope=read+write';
}

@qreeves
Copy link
Contributor

qreeves commented Apr 19, 2018

Nice catch, happy first issue!

@qreeves
Copy link
Contributor

qreeves commented Apr 19, 2018

Actually, it seems like the OAuth2 callback requires this code. Not sure how to obscure it using pure JS, so reopening this issue.

@qreeves qreeves reopened this Apr 19, 2018
@qreeves qreeves self-assigned this Apr 19, 2018
@qreeves qreeves added the needs investigating The matter is open label Apr 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs investigating The matter is open
Development

No branches or pull requests

2 participants