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

Update docker-compose.yml #25

Closed
wants to merge 11 commits into from
Closed

Conversation

mobidata-bw-user
Copy link
Member

No description provided.

Copy link
Collaborator

@hbruch hbruch left a comment

Choose a reason for hiding this comment

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

A typo and a suggestion

@@ -101,18 +101,36 @@ lamassu:
codespace: BLT
url: "https://mds.bolt.eu/gbfs/2/1352/gbfs"
language: en
authentication:
sheme: OAUTH2_CLIENT_CREDENTIALS_GRANT
Copy link
Collaborator

Choose a reason for hiding this comment

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

Instead of shemeit should be scheme, here and for all other systems

@@ -9,6 +9,11 @@ LAMASSU_BASE_URL=http://10.70.172.11:8500
LAMASSU_DB_CONNECT_CLIENT_ID=client
LAMASSU_DB_CONNECT_API_KEY=secret

# Credentials for Bolt.
LAMASSU_BOLT_API_URL=client
Copy link
Collaborator

Choose a reason for hiding this comment

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

I suggest moving LAMASSU_BOLT_API_URL with it's official value to .env, as it is no secret(?)

@mobidata-bw-user
Copy link
Member Author

bolt-authentification branch can be replace by bolt-auth branch

@mobidata-bw-user mobidata-bw-user deleted the bolt-authentification branch October 18, 2023 08:24
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.

2 participants