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

Added DataportenApi #805

Merged
merged 1 commit into from Dec 4, 2017
Merged

Added DataportenApi #805

merged 1 commit into from Dec 4, 2017

Conversation

xibriz
Copy link
Contributor

@xibriz xibriz commented Dec 3, 2017

Dataporten is a national autentication provider in Norway used by many public companies and educational institusions.

}

@Override
public Verb getAccessTokenVerb() {
Copy link
Member

Choose a reason for hiding this comment

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

POST is default, no need to override


@Override
public TokenExtractor<OAuth2AccessToken> getAccessTokenExtractor() {
return OAuth2AccessTokenJsonExtractor.instance();
Copy link
Member

Choose a reason for hiding this comment

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

this is default as well

@xibriz
Copy link
Contributor Author

xibriz commented Dec 4, 2017

@kullfar Do you want me to make the changes and do a new PR?

@kullfar
Copy link
Member

kullfar commented Dec 4, 2017

No, changes are minor. I'll make changes myself while will be merging Pull Request to the master.
But I have some troubles, running the Example. I've wrote to your email.

@xibriz
Copy link
Contributor Author

xibriz commented Dec 4, 2017

I have replied. Since Dataporten is used by so many different organization it can be a little confusing, but you want the "Feide test users" listed under "Alternative login providers for non-educational users".

@kullfar kullfar merged commit aa6098b into scribejava:master Dec 4, 2017
@kullfar
Copy link
Member

kullfar commented Dec 4, 2017

well done, I have merged it to the master

@kullfar
Copy link
Member

kullfar commented Dec 4, 2017

Thx again!

@xibriz
Copy link
Contributor Author

xibriz commented Dec 4, 2017

Thank you good sir :)

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.

None yet

2 participants