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

Getting "Could not resolve authentication method" – without a stacktrace #1110

Closed
AntonOsika opened this issue Apr 12, 2024 · 3 comments · Fixed by #1118
Closed

Getting "Could not resolve authentication method" – without a stacktrace #1110

AntonOsika opened this issue Apr 12, 2024 · 3 comments · Fixed by #1118
Assignees
Labels
bug Something isn't working triage Interesting but stale issue. Will be close if inactive for 3 more days after label added.

Comments

@AntonOsika
Copy link
Collaborator

We merged a global try catch, which I think is a bad idea.

I'm running with claude opus and when it fails I cannot see what the problem is due to the global try catch

Logs:

debug_log_file.txt

@similato87 have we used the logs so far? If not I think we should remove it or rework it.

@AntonOsika AntonOsika added bug Something isn't working triage Interesting but stale issue. Will be close if inactive for 3 more days after label added. labels Apr 12, 2024
@similato87
Copy link
Collaborator

Hi @AntonOsika , I apologize for any inconvenience this issue has caused. I understand your concerns about the global try-catch and appreciate you bringing this to my attention. Originally, we implemented this to capture exceptions related to user files more effectively, which has helped improve our logging for the new "diff" feature. We've received several valuable reports from this without needing excessive communication with our users.

However, I see how this can obscure the clarity of error reporting, especially in cases like yours with Claude Opus. I will work on a hotfix to address this by narrowing the scope of the try-catch and ensuring that stack traces are preserved for better error visibility.

@similato87 similato87 self-assigned this Apr 12, 2024
@AntonOsika
Copy link
Collaborator Author

Thanks @similato87!

Do you have a propose to how to fix it?

@similato87
Copy link
Collaborator

Hello @AntonOsika,

I have implemented a function to print the stack trace within the try-catch block, allowing us to view the trace both on the console and in the debug_log file. The attached image demonstrates successful testing. Feel free to merge the PR and run the tests again; the pytest error is unrelated to this update.

I intend to propose further enhancements once I return from my travels. Apologies for any inconvenience caused.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage Interesting but stale issue. Will be close if inactive for 3 more days after label added.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants