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

Release 1.3.0 #28

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Release 1.3.0 #28

wants to merge 16 commits into from

Conversation

hopsor
Copy link
Contributor

@hopsor hopsor commented Jan 20, 2021

No description provided.

jaimeiniesta and others added 13 commits July 3, 2019 14:06
* sets a default timeout of 10 seconds (instead of 5)

* makes params clearer in test. Thx @schurig
Mix infers the list of OTP applications the project depends on since version 1.4.
Removes obsolete instructions
The OAuth library can return also this, and not only an OAuth2.Response:

https://hexdocs.pm/oauth2/0.9.4/OAuth2.Client.html#get_token/4

This patch adds handling for both cases of error result.
* relax required Elixir version

* add support for PATCH requests

* moves timeout url to module attribute in test. Thx @seanhandley
* upgrade dependencies

* fix codeship badge
After including latest changes in a new monocle branch some e2e tests were broken when accessing stuart-api. Part of those changes were upgrading oauth version from 0.9 to 2.0. When reading the changelog we realized "application/json" serializing wasn't the default anymore so after the upgrade the token was being stored as the full json string.

With this change we explicitly set "application/json" as the content type and Jason as the serializer.

Changelog: https://github.com/scrogson/oauth2/blob/master/CHANGELOG.md#v100-2019-03-13
@hopsor hopsor changed the title Release 1.2.1 Release 1.3.0 Jan 21, 2021
Copy link

@martincabrera martincabrera left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

8 participants