Skip to content

elixir: configure via keyword list#140

Open
npry wants to merge 3 commits intomainfrom
npry/ex.config_keylist
Open

elixir: configure via keyword list#140
npry wants to merge 3 commits intomainfrom
npry/ex.config_keylist

Conversation

@npry
Copy link
Copy Markdown
Collaborator

@npry npry commented Apr 24, 2026

Use the more-ergonomic/-conventional keyword list configuration approach for Tailscale.connect, e.g.

{:ok, dev} = Tailscale.connect("key_file.json", auth_key: @my_auth_key)
# or
{:ok, dev} = Tailscale.connect(auth_key: @my_auth_key, keys: @my_keys, control_url: "https://my.headscale.com")

This also touches up docs, including removing Tailscale.Native from the generated exdoc and writing a comment noting that it should be treated as internal/perma-unstable API (there is no ability to limit module visibility on the BEAM afaik).

cc @illotum — review optional but appreciated

@npry npry force-pushed the npry/ex.config_keylist branch 5 times, most recently from 73f4fb1 to 5078cf0 Compare April 27, 2026 15:32
@npry npry marked this pull request as ready for review April 27, 2026 15:39
@npry npry requested a review from illotum April 27, 2026 15:39
Comment thread ts_elixir/native/ts_elixir/src/config.rs Outdated
@npry npry force-pushed the npry/ex.config_keylist branch from 5078cf0 to f90a763 Compare April 27, 2026 18:40
npry added 3 commits April 27, 2026 18:49
Signed-off-by: Nathan Perry <nathan@tailscale.com>
Change-Id: I6ad50d36b82f75cb0c0fc031b3bcdf0e6a6a6964
Have the `connected_client` helper indicate the IPs of the connected
client.

Signed-off-by: Nathan Perry <nathan@tailscale.com>
Change-Id: I36adfc324ff828e45bdb4d899b67e8466a6a6964
Minor updates to docs:

- Fix README code sample
- Remove `Tailscale.Native` from generated docs (`@moduledoc false`) and
make a note that it's internal and therefore perma-unstable
- Fixup reflinks

Signed-off-by: Nathan Perry <nathan@tailscale.com>
Change-Id: Icd8d68e01c90981f683886fb1dbade956a6a6964
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