applications: URI parsing bugfixes

[manicminer]

• Fix encoding of application_identifier_uri IDs to avoid injecting forward slashes into the base64 slug (closes: azuread_application_identifier_uri parsing error for hostnames exactly 10 characters long #1401)
• Loosen URI schema restrictions for application redirect URIs as the number of supported schemes has expanded and their documentation is not consolidated, making it difficult for the provider to keep up (closes: brk-multihub URIs are not accepted as SPA redirect uris #1600)

Changelog

• azuread_application - allow more URL schemes for redirect_uris properties
• azuread_application_identifier_uri - fix an encoding bug that could generate invalid resource IDs
• azuread_application_redirect_uris - allow more URL schemes for redirect_uris properties

[stephybun]

Doesn't this have to match the schema of the resource exactly? When I look at the resource schema I only see two properties

terraform-provider-azuread/internal/services/applications/application_identifier_uri_resource.go

Lines 55 to 71 in 8cead24

	"application_id": {
	Description: "The resource ID of the application to which the identifier URI should be added",
	Type: pluginsdk.TypeString,
	Required: true,
	ForceNew: true,
	ValidateFunc: stable.ValidateApplicationID,
	},
	"identifier_uri": {
	Description: "The user-defined URI or URI-like string that uniquely identifies an application within its Azure AD tenant, or within a verified custom domain if the application is multi-tenant",
	Type: pluginsdk.TypeString,
	Required: true,
	ForceNew: true,
	// Extensive validation is intentionally avoided here, as the accepted values are undocumented, vary wildly and are
	// different for each user depending on the tenant domain configuration, whether the application is used for SSO etc
	ValidateFunc: validation.StringIsNotEmpty,
	},

Unless I'm missing something? 🙈

[jackofallops]

@manicminer - This migration schema looks like it belongs to azuread_application_certificate?