Skip to content

Conversation

@iamemilio
Copy link
Contributor

Depends On #4652

What does this PR do?

  1. Standardize and sanitize error messages for conversations api
  2. Create ConfigurationError base error type for server configuration errors. This raises and logs the true error cause, but prevents the global handler from returning potentially sensitive server configuration information to API users.
  3. Updated the tests to cover the new functionality.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jan 26, 2026
@cdoern
Copy link
Collaborator

cdoern commented Jan 26, 2026

you'll need to rebase, sorry. CI was broken but is now fixed. you might also pick up some new conformance changes.

@iamemilio iamemilio force-pushed the conversations_error_message_consistency branch from 8b7afcb to dd59bc9 Compare January 26, 2026 20:13
@iamemilio iamemilio changed the title Conversations error message consistency fix: conversations error message consistency Jan 26, 2026
@iamemilio iamemilio force-pushed the conversations_error_message_consistency branch from dd59bc9 to 43b93e4 Compare January 26, 2026 20:23
@mergify
Copy link

mergify bot commented Jan 30, 2026

This pull request has merge conflicts that must be resolved before it can be merged. @iamemilio please rebase it. https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify bot added the needs-rebase label Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. needs-rebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants