Skip to content
This repository has been archived by the owner on Jul 7, 2021. It is now read-only.

#243: post user credentials in body #356

Merged
merged 1 commit into from
Nov 21, 2020
Merged

Conversation

ccostin93
Copy link
Collaborator

No description provided.

@ccostin93 ccostin93 self-assigned this Nov 21, 2020
@codecov
Copy link

codecov bot commented Nov 21, 2020

Codecov Report

Merging #356 (3fa7b80) into master (ab8d7c9) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #356   +/-   ##
=========================================
  Coverage     83.99%   83.99%           
  Complexity     1035     1035           
=========================================
  Files           175      175           
  Lines          2555     2555           
  Branches         20       20           
=========================================
  Hits           2146     2146           
  Misses          391      391           
  Partials         18       18           
Impacted Files Coverage Δ Complexity Δ
...org/proshin/finapi/accesstoken/FpAccessTokens.java 96.15% <100.00%> (ø) 7.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5b85d4...587abfe. Read the comment docs.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

proshin-roman
proshin-roman previously approved these changes Nov 21, 2020
@ccostin93 ccostin93 merged commit 7de1779 into master Nov 21, 2020
@ccostin93 ccostin93 deleted the #243-auth-urlencoded-body branch November 21, 2020 18:40
@ccostin93 ccostin93 linked an issue Nov 21, 2020 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

client/user token implementations leak credentials in logs
2 participants