Skip to content

Unsupported database type (DataStreams) by 403 http error #930

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

Merged

Conversation

kardymonds
Copy link
Collaborator

  • remove yexception
  • add try/catch to Handle(NHttp::TEvHttpProxy::TEvHttpIncomingResponse::TPtr& ev)

Copy link

github-actions bot commented Jan 11, 2024

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 4fce43e.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
59839 50614 0 13 9195 17

🔴 linux-x86_64-release-asan: some tests FAILED for commit 4fce43e.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15860 15726 0 28 103 3

vitalyisaev2
vitalyisaev2 previously approved these changes Jan 11, 2024
@kardymonds kardymonds merged commit 10f2783 into ydb-platform:main Jan 11, 2024
kardymonds added a commit to kardymonds/ydb that referenced this pull request Jan 11, 2024
…rm#930)

* Remove yexception and add try/catch

* Fix by pr comments

* Use {} init
uzhastik pushed a commit that referenced this pull request Jan 11, 2024
…stable (#944)

* Unsupported database type (DataStreams) by 403 http error (#930)

* Remove yexception and add try/catch

* Fix by pr comments

* Use {} init

* Fix Protocol and operator+
This was referenced Jan 15, 2024
@niksaveliev niksaveliev mentioned this pull request Jan 29, 2024
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
@vitstn vitstn mentioned this pull request Feb 16, 2024
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.

3 participants