-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Checkout V2: Update authorization from Basic to Bearer #5109
base: master
Are you sure you want to change the base?
Checkout V2: Update authorization from Basic to Bearer #5109
Conversation
9301f4c
to
9c64d3f
Compare
bd03d37
to
e8d04a1
Compare
def initialize(options = {}) | ||
@options = options | ||
@access_token = options[:access_token] || nil | ||
BASIC_SECRET_KEY_FORMAT = /^[0-9A-F]{8}-[0-9A-F]{4}-4[0-9A-F]{3}-[89AB][0-9A-F]{3}-[0-9A-F]{12}$/i |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you get this format from their docs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this is an assumption that I have, I am waiting for confirmation from the Checkout.com team, I am going to ask if there is any progress on this topic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, then let's waiting on this until we hear back. I'll move the ticket to waiting
requires!(options, :client_id, :client_secret) | ||
@access_token ||= setup_access_token | ||
end | ||
def initialize(options = {}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you might have accidentally added all the work from the other PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I already updated this PR
9c64d3f
to
3099951
Compare
e8d04a1
to
09ce962
Compare
3099951
to
0905d79
Compare
09ce962
to
311d29a
Compare
Start sending requests using http Auth:Bearer Spreedly reference: [ECS-3487](https://spreedly.atlassian.net/browse/ECS-3487) Unit: 66 tests, 403 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed 2821.48 tests/s, 17228.11 assertions/s Remote: 103 tests, 254 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed 0.63 tests/s, 1.56 assertions/s
0905d79
to
1fc8fa6
Compare
Start sending requests using http Auth:Bearer
Spreedly reference:
ECS-3487
Unit:
66 tests, 403 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
2821.48 tests/s, 17228.11 assertions/s
Remote:
103 tests, 254 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
0.63 tests/s, 1.56 assertions/s