Skip to content

feat(webapp): include Wire client metadata headers on backend API requests [WPB-23453]#20562

Merged
screendriver merged 1 commit intodevfrom
http-headers
Mar 3, 2026
Merged

feat(webapp): include Wire client metadata headers on backend API requests [WPB-23453]#20562
screendriver merged 1 commit intodevfrom
http-headers

Conversation

@screendriver
Copy link
Member

@screendriver screendriver commented Mar 3, 2026

TaskWPB-23453 [Web] Send platform and version number in every network request

Pull Request

Summary

Ensure backend services can reliably identify requests from the web application and associate them with the running app version for compatibility checks, observability, and policy decisions.


Security Checklist (required)

  • External inputs are validated & sanitized on client and/or server where applicable.
  • API responses are validated; unexpected shapes are handled safely (fallbacks or errors).
  • No unsafe HTML is rendered; if unavoidable, sanitization is applied and documented where it happens.
  • Injection risks (XSS/SQL/command) are prevented via safe APIs and/or escaping.

Accessibility (required)

Standards Acknowledgement (required)


Screenshots or demo (if the user interface changed)

Notes for reviewers

  • Trade-offs:
  • Follow-ups (linked issues):
  • Linked PRs (e.g. web-packages):

…uests

Ensure backend services can reliably identify requests from the web
application and associate them with the running app version for
compatibility checks, observability, and policy decisions.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 9
  • Failed: 0
  • Skipped: 3
  • 🔁 Flaky: 2
  • 📊 Total: 14
  • Total Runtime: 79.8s (~ 1 min 20 sec)
specs/CriticalFlow/conversationManagement-TC-8636.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Conversation Management (tags: TC-8636, crit-flow-web)
specs/CriticalFlow/joinTeam-TC-8635.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ New person joins team and sets up device (tags: TC-8635, crit-flow-web)

@screendriver screendriver added this pull request to the merge queue Mar 3, 2026
@arjita-mitra
Copy link
Contributor

LGTM 🚀

Merged via the queue into dev with commit e693bfe Mar 3, 2026
53 checks passed
@screendriver screendriver deleted the http-headers branch March 3, 2026 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants