Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Major: rename default container name to localstack-main #892

Merged
merged 5 commits into from
Nov 9, 2023

Conversation

simonrw
Copy link
Contributor

@simonrw simonrw commented Nov 6, 2023

Motivation

Underscores are not allowed in URLs, but hyphens are. When addressing the LocalStack container over a docker network, the container name resolves to the IP address of the container. This is a problem in particular for boto3 which refuses to connect.

Changes

This PR updates usages of localstack_main to localstack-main.

@simonrw simonrw self-assigned this Nov 6, 2023
Copy link

github-actions bot commented Nov 6, 2023

🎊 PR Preview has been successfully built and deployed to https://localstack-docs-preview-pr-892.surge.sh 🎊

@simonrw simonrw force-pushed the update-default-container-name branch from 7036c46 to a6c0e68 Compare November 6, 2023 11:29
@dominikschubert dominikschubert mentioned this pull request Nov 8, 2023
15 tasks
@dominikschubert dominikschubert self-requested a review November 8, 2023 22:28
@simonrw simonrw marked this pull request as ready for review November 8, 2023 22:31
@dominikschubert dominikschubert self-requested a review November 8, 2023 23:12
Copy link
Member

@dominikschubert dominikschubert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@simonrw Before merging please look over the added docker-compose sample and adjust it if you see any issues.

@alexrashed alexrashed force-pushed the update-default-container-name branch from 0344514 to 37b5c12 Compare November 9, 2023 06:13
@simonrw simonrw merged commit 74a45b0 into release/v3.0 Nov 9, 2023
@simonrw simonrw mentioned this pull request Nov 9, 2023
@cabeaulac
Copy link
Contributor

I think the Motivation here should say "underscores are not allowed...". Hyphens are allowed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants