Project

General

Profile

Bug #52738

notifications: http endpoints with one trailing slash are considered malformed

Added by Yuval Lifshitz almost 2 years ago. Updated 11 months ago.

Status:
Resolved
Priority:
Normal
Target version:
-
% Done:

0%

Source:
Support
Tags:
easy first bug backport_processed
Backport:
pacific, octopus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

following command is considered as malformed URL:

aws --endpoint-url http://localhost:8000 sns create-topic --name=mytopic1 --attributes='{"push-endpoint": "http://localhost:8080/"}'

while the following are OK:
aws --endpoint-url http://localhost:8000 sns create-topic --name=mytopic1 --attributes='{"push-endpoint": "http://localhost:8080"}'
aws --endpoint-url http://localhost:8000 sns create-topic --name=mytopic1 --attributes='{"push-endpoint": "http://localhost:8080/something"}'

to fix this issue, the regular expression for the "path" in rgw_url.cc should be modified to allow for "http://localhost:8080/" as a valid URL.


Related issues

Copied to rgw - Backport #53078: octopus: notifications: http endpoints with one trailing slash are considered malformed Resolved
Copied to rgw - Backport #53079: pacific: notifications: http endpoints with one trailing slash are considered malformed Resolved

History

#1 Updated by Yuval Lifshitz almost 2 years ago

a test case should be added to: test_rgw_url.cc

#2 Updated by Yuval Lifshitz almost 2 years ago

  • Assignee set to Kalpesh Pandya

#3 Updated by Casey Bodley almost 2 years ago

  • Status changed from New to Pending Backport
  • Pull request ID set to 43528

#4 Updated by Backport Bot almost 2 years ago

  • Copied to Backport #53078: octopus: notifications: http endpoints with one trailing slash are considered malformed added

#5 Updated by Backport Bot almost 2 years ago

  • Copied to Backport #53079: pacific: notifications: http endpoints with one trailing slash are considered malformed added

#6 Updated by Backport Bot about 1 year ago

  • Tags changed from easy first bug to easy first bug backport_processed

#7 Updated by Konstantin Shalygin 11 months ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF