Currently the push notifications service is expecting a configuration for proxy like:
proxy:
host: <host>
port: <port>
protocol: <protocol>For consistency with the rest of the apps we should adapt this in order to accept a URL like <protocol>//<host>:<port>
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Adapt proxy config to be consistent with other services. | mediawiki/services/push-notifications | master | +61 -15 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Jgiannelos | T250452 New Service Request: Wikimedia push notification service | |||
| Resolved | jijiki | T256973 Deploy push-notifications service to Kubernetes | |||
| Resolved | MSantos | T261635 Test push-notifications service in the staging environment | |||
| Resolved | Jgiannelos | T262444 Adapt proxy config handling to accept a proxy URL instead of a dict |
Change 626352 had a related patch set uploaded (by Jgiannelos; owner: Jgiannelos):
[mediawiki/services/push-notifications@master] Adapt proxy config to be consistent with other services.
Change 626352 merged by jenkins-bot:
[mediawiki/services/push-notifications@master] Adapt proxy config to be consistent with other services.