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

Use custom token endpoint url from AuthenticationCredentials #1433

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

mykolamalik
Copy link
Collaborator

The token provider read from a credentials file in the method AuthenticationCredentials::ReadFromStream() was not used for the authentication request.

Relates-To: OLPEDGE-2834

@mykolamalik mykolamalik force-pushed the bug/olpedge-2834 branch 2 times, most recently from 1eec01f to 988ed3e Compare September 18, 2023 15:21
@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Merging #1433 (803071b) into master (9fb70ea) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 803071b differs from pull request most recent head 998bfa7. Consider uploading reports for the commit 998bfa7 to get more accurate results

@@            Coverage Diff             @@
##           master    #1433      +/-   ##
==========================================
- Coverage   79.96%   79.95%   -0.00%     
==========================================
  Files         339      339              
  Lines       13042    13044       +2     
  Branches     1415     1415              
==========================================
+ Hits        10428    10429       +1     
  Misses       2042     2042              
- Partials      572      573       +1     
Files Changed Coverage Δ
olp-cpp-sdk-authentication/src/Settings.cpp 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

The token provider read from a credentials file in the method
`AuthenticationCredentials::ReadFromStream()` was not used for
the authentication request.

Relates-To: OLPEDGE-2834

Signed-off-by: Mykola Malik <ext-mykola.malik@here.com>
@ystefinko
Copy link
Collaborator

Gitlab-bot - Internal E2E verification pipeline FAILED!

@mykolamalik mykolamalik merged commit 58f9566 into master Sep 19, 2023
9 checks passed
@mykolamalik mykolamalik deleted the bug/olpedge-2834 branch September 19, 2023 10:06
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.

4 participants