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

Logging from Enso includes weird class name #11262

Closed
radeusgd opened this issue Oct 5, 2024 · 1 comment · Fixed by #11519
Closed

Logging from Enso includes weird class name #11262

radeusgd opened this issue Oct 5, 2024 · 1 comment · Fixed by #11519
Assignees
Labels
-compiler -libs Libraries: New libraries to be implemented

Comments

@radeusgd
Copy link
Member

radeusgd commented Oct 5, 2024

I was just testing some logging in #11261 and here's an excerpt:

INFO: Access token expires at 2024-10-05 16:20:00[Z], so we are refreshing it.
paź 05, 2024 7:20:31 PM com.oracle.truffle.host.HostMethodDesc$SingleMethod$MHBase invokeHandle
WARNING: Refresh token request failed with status 401 Unauthorized: Invalid refresh token..
paź 05, 2024 7:20:31 PM com.oracle.truffle.host.HostMethodDesc$SingleMethod$MHBase invokeHandle
INFO: Access token expires at 2024-10-05 16:20:00[Z], so we are refreshing it.

The class name com.oracle.truffle.host.HostMethodDesc$SingleMethod$MHBase invokeHandle is not really expected.

I'd rather expect something along Standard.Base.Enso_Cloud.Internal.Authentication there, since that's the class name we use getLogger for:

Any.log_message self ~message:Text level:Log_Level=Log_Level.Info =
    type_name = Meta.get_qualified_type_name self
    logger = Logger.getLogger type_name
@enso-bot
Copy link

enso-bot bot commented Nov 13, 2024

Hubert Plociniczak reports a new STANDUP for the provided date (2024-11-11):

Progress: Figured out a way to test stdlib logging. CI appears to be a bit off, investigating. Otherwise, just trying to figure out #11389. It should be finished by 2024-11-11.

Next Day: Next day I will be working on the #11389 task. Continue investigating the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-compiler -libs Libraries: New libraries to be implemented
Projects
Status: 🟢 Accepted
Development

Successfully merging a pull request may close this issue.

3 participants