search

froala / wysiwyg-editor-php-sdk

PHP SDK to ease the integration of Froala WYSIWYG Editor on server side.
https://www.froala.com/wysiwyg-editor
MIT License
42 stars 26 forks source link

Correct URL for other regions #27

Open jacovdbergh opened 4 years ago

jacovdbergh commented 4 years ago

Buckets other than us-east-1 incorrectly had s3- prepended, where it should have been s3.

https://s3-af-south-1.amazonaws.com/bucket-name

which should be

https://s3.af-south-1.amazonaws.com/bucket-name

jacovdbergh commented 4 years ago

@stefanneculai any feedback here?

vivek-accolite-801 commented 1 year ago

@jacovdbergh Sorry for the delayed response. Both of the URL formats are support by S3, I think. Though I think we keep it consistent - will include this one.

Thanks

vivek-accolite-801 commented 1 year ago

@harasunu-narayan - Added as a part of https://github.com/froala/wysiwyg-editor-php-sdk/pull/30

jacovdbergh commented 1 year ago

Haha forgot about this, but thanks :D