feat: Allow server side configuration of pubsub#26
feat: Allow server side configuration of pubsub#26pedromarceOAT wants to merge 1 commit intoNeoScript:mainfrom
Conversation
|
This is very important to make this image useful, especially when using in ephemeral testing environments using tools like garden.io please do merge this as soon as possible. Thank you. |
|
@NeoScript @AlbertoMonteiro any chance we could merge this? |
|
It would be great to merge this. We need to be able to specify FYI @NeoScript 😊 |
|
@NeoScript @AlbertoMonteiro What are the chances of this being merged? I need this feature. |
I can't do anything about that. @NeoScript is the repo owner, only he can do it. |
|
@NeoScript, help us. Please 🙏. |
There was a problem hiding this comment.
I would like to leave a suggestion to add also along with variavel of environment pubsub-emulator the project_id. It would make the setup of the environment more flexible and already have these two settings reduces the cognitive load.
To end could add in the various docker-compose environments
environment:
PUBSUB_EMULATOR_HOST: localhost:8681 #https://idp-remot.paas:8085
PROJECT_ID: test-project| export const environment = { | ||
| production: true | ||
| production: true, | ||
| pubsubEmulatorHost: process.env['PUBSUB_EMULATOR_HOST'] ?? "http://localhost:8681" |
There was a problem hiding this comment.
praise (non-blocking):
Can we add the PROJECT_ID along with the pubsub host?
projectId: process.env['PROJECT_ID'] ?? "test-project"| private http = inject(HttpClient); | ||
|
|
||
| public _currentHost$ = new BehaviorSubject<string>("http://localhost:8681") | ||
| public _currentHost$ = new BehaviorSubject<string>(environment.pubsubEmulatorHost) |
There was a problem hiding this comment.
praise (non-blocking):
Can we add the PROJECT_ID along with the pubsub host? Suggestion line 36.
this._projectList.next([environment.projectId])| "builder": "angular-server-side-configuration:ngsscbuild", | ||
| "options": { | ||
| "additionalEnvironmentVariables": [ | ||
| "PUBSUB_EMULATOR_HOST" |
There was a problem hiding this comment.
praise (non-blocking):
Can we add the PROJECT_ID along with the pubsub host?
PROJECT_ID| production: false | ||
| production: false, | ||
| pubsubEmulatorHost: process.env['PUBSUB_EMULATOR_HOST'] ?? "http://localhost:8681" | ||
| }; |
There was a problem hiding this comment.
praise (non-blocking):
Can we add the PROJECT_ID along with the pubsub host?
projectId: process.env['PROJECT_ID'] ?? "test-project"
|
Hi, |
|
I think the problem is not as simple as providing an env variable.The website doesn't have a way to pull data from backend, it's all purely frontend. I solved this issue manually by using multi stage builds and manually updating the endpoint in the build process. It's definitely not the right way of doing this but it unblocked me. |
|
I understand. Thank you for explaining. |
9 participants
Allow external configuration to be provided to configure URL of the pubsub emulator.
Needs to be provided as an environment variable in the docker container "PUBSUB_EMULATOR_HOST"
Should solve this issue #25