Skip to content

Comments

Update bas-external-system.mdx with .env configuration#2286

Open
drvup wants to merge 1 commit intoSAP:mainfrom
drvup:patch-3
Open

Update bas-external-system.mdx with .env configuration#2286
drvup wants to merge 1 commit intoSAP:mainfrom
drvup:patch-3

Conversation

@drvup
Copy link
Contributor

@drvup drvup commented Oct 21, 2025

I did run the 2nd time into this small tiny and time-consuming issue. So, for the next time, everyone should be aware of it :)

What Has Changed?

The .env file needs to have the variables within 1 line, without breaks.

I did run the 2nd time into this small tiny and time-consuming issue. So, for the next time, everyone should be aware of it :)
@marikaner
Copy link
Contributor

Hey @drvup, it seems this was missed for quite a while. I used to think that .env only accepts values on one line, however, I just learned last week, that this is not the case. I believe our example is incorrect though, because it uses " for the inner strings and the outer string. I think this should also work:

destinations='[{
"name": "<destinationName>",
"url": "http://<destinationName>.dest",
"proxyConfiguration":{
      "host": "127.0.0.1",
      "port": 8887,
      "protocol":"http"
  }
}]'

I would update the example to reflect that instead. Any objections?

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