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

[Fix] Change Auth method log to debug instead of info #371

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zashroof
Copy link

@zashroof zashroof commented Oct 29, 2024

Change info log line about Auth method to debug to avoid spamming the log file. Currently this is the only log in the sdk that directly writes to info

image

Tests

not tested

Signed-off-by: Ashraf Saleh a.saleh@celonis.com

@mgyucht mgyucht changed the title Change Auth method log to debug instead of info [Fix] Change Auth method log to debug instead of info Oct 30, 2024
@zashroof zashroof force-pushed the auth-method-log-debug branch 2 times, most recently from 518d585 to a0f9113 Compare October 30, 2024 15:52
@eng-dev-ecosystem-bot
Copy link
Collaborator

Test Details: go/deco-tests/11617989674

@zashroof zashroof force-pushed the auth-method-log-debug branch from 192bafa to a591332 Compare October 31, 2024 20:42
@zashroof zashroof closed this Nov 1, 2024
@zashroof zashroof force-pushed the auth-method-log-debug branch from a591332 to a1460c5 Compare November 1, 2024 13:56
Signed-off-by: Ashraf Saleh <a.saleh@celonis.com>
@zashroof zashroof reopened this Nov 1, 2024
Copy link

github-actions bot commented Nov 1, 2024

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-java

Inputs:

  • PR number: 371
  • Commit SHA: 497c44316492e98c8dd2fac3803aed400ced2763

Checks will be approved automatically on success.

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