Skip to content

Commit c05db92

Browse files
authored
Merge pull request #8 from benchling/fix-readme
Fix misplaced info in the README for chem-sync-local-flask
2 parents 4801f64 + 2ffde67 commit c05db92

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

examples/chem-sync-local-flask/README.md

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,12 @@ pbpaste > .client_secret
131131
132132
You'll then need to restart _just_ the `benchling-app` Docker service to pick up the changes:
133133

134+
```bash
135+
docker-compose up -d
136+
```
137+
138+
If you restart both containers, be sure to update your App in Benchling with the new webhook URL from localtunnel.
139+
134140
### Setting Client ID
135141

136142
Our App needs a Client ID to pair with the Client Secret for authentication to Benchling. In this case, we've created our
@@ -155,12 +161,12 @@ for your App. For example:
155161
CLIENT_ID=42a0cd39-0543-4dd2-af02-a866c97f0c4d
156162
```
157163

158-
```
164+
Restart the `benchling-app` Docker container to pick up the environment changes.
165+
166+
```bash
159167
docker-compose up -d
160168
```
161169

162-
If you restart both containers, be sure to update your App in Benchling with the new webhook URL from localtunnel.
163-
164170
> ⚠️ **Security Note:** In production, store the secret with a secure solution such as a secrets store (AWS Secrets Manager, as an example) or, if storing programmatically, encrypted using app-layer encryption. Avoid placing it in plaintext anywhere in code or configuration.
165171
166172
### Create App Registry Dependencies

0 commit comments

Comments
 (0)