remove HasLicense field from Options and implement HasLicense method in Entitlements

kacpersaw · kacpersaw · commit fa715a9f7f81 · 2025-08-13T12:08:07.000Z
diff --git a/coderd/coderd.go b/coderd/coderd.go
@@ -180,8 +180,6 @@ type Options struct {
 	// Entitlements can come from the enterprise caller if enterprise code is
 	// included.
 	Entitlements *entitlements.Set
-	// HasLicense indicates if a license is installed.
-	HasLicense bool
 	// PostAuthAdditionalHeadersFunc is used to add additional headers to the response
 	// after a successful authentication.
 	// This is somewhat janky, but seemingly the only reasonable way to add a header
diff --git a/coderd/entitlements/entitlements.go b/coderd/entitlements/entitlements.go
@@ -161,3 +161,9 @@ func (l *Set) Errors() []string {
 	defer l.entitlementsMu.RUnlock()
 	return slices.Clone(l.entitlements.Errors)
 }
+
+func (l *Set) HasLicense() bool {
+	l.entitlementsMu.RLock()
+	defer l.entitlementsMu.RUnlock()
+	return l.entitlements.HasLicense
+}
diff --git a/enterprise/coderd/coderd.go b/enterprise/coderd/coderd.go
@@ -739,8 +739,6 @@ func (api *API) updateEntitlements(ctx context.Context) error {
 			return codersdk.Entitlements{}, err
 		}
 
-		api.HasLicense = reloadedEntitlements.HasLicense
-
 		if reloadedEntitlements.RequireTelemetry && !api.DeploymentValues.Telemetry.Enable.Value() {
 			api.Logger.Error(ctx, "license requires telemetry enabled")
 			return codersdk.Entitlements{}, entitlements.ErrLicenseRequiresTelemetry
@@ -965,7 +963,7 @@ func (api *API) CheckBuildUsage(ctx context.Context, store database.Store, templ
 	// When unlicensed, we need to check that we haven't breached the managed agent
 	// limit.
 	// Unlicensed deployments are allowed to use unlimited managed agents.
-	if api.HasLicense {
+	if api.Entitlements.HasLicense() {
 		managedAgentLimit, ok := api.Entitlements.Feature(codersdk.FeatureManagedAgentLimit)
 		if !ok || !managedAgentLimit.Enabled || managedAgentLimit.Limit == nil || managedAgentLimit.UsagePeriod == nil {
 			return wsbuilder.UsageCheckResponse{
