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

Improved logged parameters in SsoCallbackServlet request #3244

Merged
merged 1 commit into from
Feb 17, 2021

Conversation

gbarbon
Copy link
Contributor

@gbarbon gbarbon commented Feb 17, 2021

This PR adds a couple of parameters, state and session_state, to the logger in the doGet method of the SsoCallbackServlet.

Related Issue
n/a

Description of the solution adopted
n/a

Screenshots
n/a

Any side note on the changes made
n/a

Signed-off-by: Gianluca Barbon <gianluca.barbon@eurotech.com>
@gbarbon gbarbon added the Enhancement This PR/Issue improves an part of Kapua label Feb 17, 2021
@gbarbon gbarbon requested a review from lorthirk February 17, 2021 15:08
@gbarbon gbarbon self-assigned this Feb 17, 2021
@codecov
Copy link

codecov bot commented Feb 17, 2021

Codecov Report

Merging #3244 (3b824e5) into develop (ffd9a30) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #3244      +/-   ##
=============================================
+ Coverage      59.36%   59.39%   +0.02%     
- Complexity      3014     3018       +4     
=============================================
  Files           1273     1273              
  Lines          27038    27038              
  Branches        2346     2346              
=============================================
+ Hits           16052    16059       +7     
+ Misses         10089    10082       -7     
  Partials         897      897              
Impacted Files Coverage Δ Complexity Δ
...ua/broker/core/listener/DeviceMessageListener.java 60.46% <0.00%> (-6.98%) 0.00% <0.00%> (ø%)
...ce/management/commons/call/DeviceCallExecutor.java 62.79% <0.00%> (-4.66%) 0.00% <0.00%> (ø%)
...atch/persistence/jpa/JpaExecutionInstanceData.java 87.80% <0.00%> (-2.44%) 22.00% <0.00%> (-1.00%)
.../broker/core/plugin/KapuaSecurityBrokerFilter.java 76.85% <0.00%> (+0.28%) 0.00% <0.00%> (ø%)
...plugin/authentication/UserAuthenticationLogic.java 83.03% <0.00%> (+1.78%) 0.00% <0.00%> (ø%)
...nnection/internal/DeviceConnectionServiceImpl.java 69.31% <0.00%> (+2.27%) 17.00% <0.00%> (+1.00%)
.../DeviceManagementOperationRegistryServiceImpl.java 69.64% <0.00%> (+3.57%) 14.00% <0.00%> (+2.00%)
...ages/job/DevicePackageDownloadTargetProcessor.java 100.00% <0.00%> (+5.00%) 9.00% <0.00%> (+1.00%)
...obDeviceManagementOperationManagerServiceImpl.java 79.26% <0.00%> (+6.09%) 14.00% <0.00%> (+1.00%)

@Coduz Coduz merged commit fa22688 into eclipse-kapua:develop Feb 17, 2021
@gbarbon gbarbon deleted the feature-improvingSsoLog branch February 17, 2021 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement This PR/Issue improves an part of Kapua
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants