Skip to content

Conversation

@wivaku
Copy link
Contributor

@wivaku wivaku commented Dec 28, 2025

Tailscale now has tsidp: Tailscale OpenID Connect (OIDC) Identity Provider
Instructions on how to set up tsidp for local Tailscale network:
https://github.com/tailscale/tsidp

Copy link
Member

@atymic atymic left a comment

Choose a reason for hiding this comment

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

looks good, few minor bits

"SocialiteProviders\\Tailscale\\": ""
}
}
} No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

newline

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I copied composer.json from other providers, which don't have newline. What's the policy?

Copy link
Member

Choose a reason for hiding this comment

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

Should have a newline

"docs": "https://socialiteproviders.com/tailscale"
},
"require": {
"php": "^8.0",
Copy link
Member

Choose a reason for hiding this comment

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

8.4 +

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I copied composer.json from other providers, e.g. Okta. Most of them require 8.0+.
Is there anything 8.4 specific in the code?

Copy link
Member

Choose a reason for hiding this comment

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

Sorry, 8.3 is ok. we only support supported php vers

@atymic
Copy link
Member

atymic commented Jan 2, 2026

Can you please fix the lint error?

@atymic atymic merged commit 0f28fe6 into SocialiteProviders:master Jan 2, 2026
3 checks passed
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