Skip to content

Add Basic Auth to Signal Messenger #41589

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 10 commits into
base: next
Choose a base branch
from
Open

Add Basic Auth to Signal Messenger #41589

wants to merge 10 commits into from
+30 −0

Conversation

@CodeShellDev
Copy link

@CodeShellDev CodeShellDev commented Nov 3, 2025

Proposed change

This PR adds Basic Auth documentation to the Signal Messenger Integration.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@home-assistant home-assistant bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Nov 3, 2025
@netlify
Copy link

netlify bot commented Nov 3, 2025 •
edited
Loading

✅ Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 204e71f
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/690b22fdfe73640008d3f9b9
😎 Deploy Preview https://deploy-preview-41589--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@CodeShellDev CodeShellDev marked this pull request as draft November 3, 2025 16:54
@CodeShellDev CodeShellDev mentioned this pull request Nov 3, 2025
Open
21 tasks
@CodeShellDev CodeShellDev marked this pull request as ready for review November 3, 2025 16:58
@CodeShellDev
Copy link
Author

CodeShellDev commented Nov 3, 2025

Hey there,
I would like to get some guidance on how adding a nested configuration attribute in the {% configuration %} works?

Thanks in advance!

@CodeShellDev CodeShellDev marked this pull request as draft November 3, 2025 17:07
@CodeShellDev
Copy link
Author

CodeShellDev commented Nov 3, 2025

Hey there, I would like to get some guidance on how adding a nested configuration attribute in the {% configuration %} works?

Thanks in advance!

Seems to be rendering correctly.

@CodeShellDev CodeShellDev marked this pull request as ready for review November 3, 2025 17:11
@CodeShellDev
Copy link
Author

CodeShellDev commented Nov 3, 2025

Could I add secured-signal-api as an example for a Basic Auth Proxy? Or is this not permitted / wanted?

CodeShellDev
CodeShellDev commented Nov 3, 2025
View reviewed changes
c0ffeeca7
c0ffeeca7 requested changes Nov 4, 2025
View reviewed changes
Copy link
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tiny tweaks

@home-assistant
Copy link

home-assistant bot commented Nov 4, 2025

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@home-assistant home-assistant bot marked this pull request as draft November 4, 2025 05:37
CodeShellDev and others added 5 commits November 4, 2025 06:45
@CodeShellDev @c0ffeeca7
signal_messenger.markdown update
b586aa4 
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
@CodeShellDev @c0ffeeca7
signal_messenger.markdown update
93bf457 
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
@CodeShellDev @c0ffeeca7
signal_messenger.markdown update
897e989 
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
@CodeShellDev @c0ffeeca7
signal_messenger.markdown update
cda44fc 
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
@CodeShellDev
Merge branch 'next' into signal-messenger-basic-auth
acb4c50
@CodeShellDev CodeShellDev marked this pull request as ready for review November 4, 2025 05:47
@home-assistant home-assistant bot requested a review from c0ffeeca7 November 4, 2025 05:47
CodeShellDev
CodeShellDev commented Nov 4, 2025
View reviewed changes
source/_integrations/signal_messenger.markdown Outdated

### Basic Auth

If Signal Messenger is running behind a Proxy with Basic Auth requirements, you need to add `auth` to your config and specify the username and password respectively.
Copy link
Author

@CodeShellDev CodeShellDev Nov 4, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could I add secured-signal-api, as an example?

It is a Proxy made specifically for Signal Messenger REST API

Copy link
Author

@CodeShellDev CodeShellDev Nov 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or maybe add to a separate related projects section?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@c0ffeeca7 c0ffeeca7 Awaiting requested review from c0ffeeca7

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch second-opinion-wanted

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@CodeShellDev @c0ffeeca7