Skip to content

Object Storage - Patch s3 compliancy #7983

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
merged 20 commits into from
Jun 17, 2025
Merged

Conversation

lason-ovh
Copy link
Contributor

@lason-ovh lason-ovh commented Jun 10, 2025

What type of Pull Request is this?

  • Fix

Description

There were missing http verbs that we normally support in pre-signed urls. Also, we added a new sub-category for POST because it is a feature on its own and to eliminate confusion.

Mandatory information

  • This Pull Request can be merged as soon as possible AFTER reviews from @agoude
  • This Pull Request content should be replicated for the US OVHcloud documentation : YES

ajout des verbes http manquants
added missing supported http verbs
@Y0Coss Y0Coss changed the title Lso patch doc s3 compliancy Object Storage - Patch s3 compliancy Jun 10, 2025
@DoStupidThingsFasterWithMoreEnergy
Copy link
Contributor

DoStupidThingsFasterWithMoreEnergy commented Jun 11, 2025

To be consistent with the issue mentioned here: #7874, perhaps replace the term Pre-signed urls with Presigned urls, which is the term most commonly used by AWS. This would allow this guide to be displayed in the search results for the presign function, which is not mentioned in this guide.

pre-signed_query

presigned instead of pre-signed
presigned instead of pre-signed
@lason-ovh
Copy link
Contributor Author

I replaced pre-signed with presigned according to preconisations

lason-ovh added 12 commits June 13, 2025 17:48
ajout des screenshots FR
changed title indentation for Offsite Replication
modification de l'indentation du titre pour Offsite repli
…ation/images/replication-rule-creation-FR.PNG

deleted rule creation FR
…ation/images/replication-rules-delete-FR.png

screenshot has not supported edit functionality
ajout de la partie UI pour configurer une règle de répli
/!\ il manque encore un screenshot
added instructions to create a replication rule using the UI
/!\ screenshots in EN have yet to be added
added rule creation screenshot in FR
ajout du screenshot manquant
added EN screeshots
added missing screenshots
utiisation de panneau de ctrl au lieu de web ui
using Ctrl panel instead of web ui
@lason-ovh lason-ovh requested a review from agoude June 16, 2025 13:06
@Y0Coss Y0Coss merged commit 1113715 into develop Jun 17, 2025
@Y0Coss Y0Coss deleted the lso_patch_doc_s3_compliancy branch June 17, 2025 08:43
@Y0Coss
Copy link
Collaborator

Y0Coss commented Jun 17, 2025

Thank you for your contribution @lason-ovh

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.

4 participants