Skip to content
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

Add Dinahosting S3 Storage Profile #124

Merged
merged 2 commits into from
Jan 8, 2025
Merged

Conversation

eduisnot
Copy link
Contributor

@eduisnot eduisnot commented Jan 8, 2025

Profile for Dinahosting S3 Storage added. Thanks.

@dkocher dkocher linked an issue Jan 8, 2025 that may be closed by this pull request
@dkocher dkocher self-assigned this Jan 8, 2025
@dkocher dkocher self-requested a review January 8, 2025 16:52
@dkocher
Copy link
Contributor

dkocher commented Jan 8, 2025

I suppose the service does only support a single region. Can you add a Region and Regions property to the profile like

        <key>Region</key>
        <string>ap-south-1</string>
        <key>Regions</key>
        <array>
            <string>ap-south-1</string>
        </array>

If the default region is not documented you may look it up in Info → S3 → Location

@eduisnot
Copy link
Contributor Author

eduisnot commented Jan 8, 2025

Done! Thanks for the tip about where to look up. The service (per their docummentation and support) 'doesn't have regions', and CyberDuck says Info → S3 → Location: default

So added 'default' and tried again, working fine. Thanks.

@dkocher
Copy link
Contributor

dkocher commented Jan 8, 2025

Done! Thanks for the tip about where to look up. The service (per their docummentation and support) 'doesn't have regions', and CyberDuck says Info → S3 → Location: default

So added 'default' and tried again, working fine. Thanks.

Can you confirm creating a new bucket using File → New Folder… works with this configuration.

@eduisnot
Copy link
Contributor Author

eduisnot commented Jan 8, 2025

Just tried, working fine

image

And replicated in the service provider web explorer:

image

Thanks

@dkocher dkocher merged commit f25378b into iterate-ch:main Jan 8, 2025
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.

Add connection profile for Dinahosting S3 Storage
2 participants