object: Wrap CheckAPE errors to status errors #1040

Merged

fyrchik merged 1 commit from aarifullin/frostfs-node:fix/status_ape_errors into master

2024-03-14 07:34:08 +00:00

aarifullin commented

2024-03-12 14:11:46 +00:00

Member

All methods should wrap CheckAPE error, if it occurs, to status error.

* All methods should wrap CheckAPE error, if it occurs, to status error.

aarifullin force-pushed fix/status_ape_errors from bfc594f5d5 to 457116494d

2024-03-12 14:12:40 +00:00

Compare

fyrchik requested review from storage-core-committers 2024-03-13 08:48:51 +00:00

fyrchik requested review from storage-core-developers 2024-03-13 08:48:51 +00:00

fyrchik approved these changes 2024-03-13 08:48:56 +00:00

fyrchik commented

2024-03-13 08:49:06 +00:00

Owner

Is it related to any bug or issue?

acid-ant approved these changes 2024-03-13 08:53:39 +00:00

aarifullin commented

2024-03-13 09:13:47 +00:00

Author

Member

Is it related to any bug or issue?

If responses from remote Head request are not wrapped to status, then a performing node, that has requested remote node, returns Object Not Found error because the status is default.
Actually, I already tried to fix that but unintentionally skipped this for some methods (used primitive way - replacing)

> Is it related to any bug or issue? If responses from remote `Head` request are not wrapped to status, then a performing node, that has requested remote node, returns `Object Not Found` error because the status is default. Actually, I already tried to fix that but unintentionally skipped this for some methods (used primitive way - replacing)

fyrchik merged commit d7be70e93f into master

2024-03-14 07:34:08 +00:00

fyrchik referenced this pull request from a commit

2024-03-14 07:34:10 +00:00

[#1040] object: Wrap CheckAPE errors to status errors