Refactor as per PR comments

Author: Damans227

api/src/main/java/org/apache/cloudstack/auth/UserOAuth2Authenticator.java

@@ -46,13 +46,13 @@ public interface UserOAuth2Authenticator extends Adapter {
 
     /**
      * Verifies if the logged in user is valid for a specific domain
-     * @return returns true if its valid user
+     * @return true if it's a valid user, otherwise false
      */
     boolean verifyUser(String email, String secretCode, Long domainId);
 
     /**
-     * Verifies the code provided by provider and fetches email for a specific domain
-     * @return returns email
+     * Verifies the secret code provided by provider and fetches email for a specific domain
+     * @return email for the specified domain
      */
     String verifyCodeAndFetchEmail(String secretCode, Long domainId);
 

plugins/user-authenticators/oauth2/src/main/java/org/apache/cloudstack/oauth2/OAuth2AuthManager.java

@@ -34,7 +34,7 @@ public interface OAuth2AuthManager extends PluggableAPIAuthenticator, PluggableS
     Long GLOBAL_DOMAIN_ID = -1L;
 
     public static ConfigKey<Boolean> OAuth2IsPluginEnabled = new ConfigKey<Boolean>("Advanced", Boolean.class, "oauth2.enabled", "false",
-            "Indicates whether OAuth plugin is enabled or not. Can be configured at domain level.", true, ConfigKey.Scope.Domain);
+            "Indicates whether OAuth plugin is enabled or not. This can be configured at domain level.", true, ConfigKey.Scope.Domain);
     public static final ConfigKey<String> OAuth2Plugins = new ConfigKey<String>("Advanced", String.class, "oauth2.plugins", "google,github",
             "List of OAuth plugins", true);
     public static final ConfigKey<String> OAuth2PluginsExclude = new ConfigKey<String>("Advanced", String.class, "oauth2.plugins.exclude", "",

plugins/user-authenticators/oauth2/src/main/java/org/apache/cloudstack/oauth2/api/command/ListOAuthProvidersCmd.java

@@ -64,11 +64,11 @@ public class ListOAuthProvidersCmd extends BaseListCmd implements APIAuthenticat
     private String provider;
 
     @Parameter(name = ApiConstants.DOMAIN_ID, type = CommandType.UUID, entityType = DomainResponse.class,
-            description = "List OAuth providers for a specific domain. Use -1 for global providers only.")
+            description = "List OAuth providers for a specific domain. Use -1 for global providers only.", since = "4.23.0")
     private Long domainId;
 
     @Parameter(name = ApiConstants.DOMAIN, type = CommandType.STRING,
-            description = "Domain path for domain-specific OAuth provider lookup")
+            description = "Domain path for domain-specific OAuth provider lookup", since = "4.23.0")
     private String domainPath;
 
     /////////////////////////////////////////////////////

plugins/user-authenticators/oauth2/src/main/java/org/apache/cloudstack/oauth2/api/command/RegisterOAuthProviderCmd.java

@@ -60,7 +60,7 @@ public class RegisterOAuthProviderCmd extends BaseCmd {
     private String redirectUri;
 
     @Parameter(name = ApiConstants.DOMAIN_ID, type = CommandType.UUID, entityType = DomainResponse.class,
-            description = "Domain ID for domain-specific OAuth provider. If not provided, registers as global provider")
+            description = "Domain ID for domain-specific OAuth provider. If not provided, registers as global provider", since = "4.23.0")
     private Long domainId;
 
     @Parameter(name = ApiConstants.DETAILS, type = CommandType.MAP,

plugins/user-authenticators/oauth2/src/main/java/org/apache/cloudstack/oauth2/api/command/VerifyOAuthCodeAndGetUserCmd.java

@@ -60,11 +60,11 @@ public class VerifyOAuthCodeAndGetUserCmd extends BaseListCmd implements APIAuth
     private String secretCode;
 
     @Parameter(name = ApiConstants.DOMAIN_ID, type = CommandType.UUID, entityType = DomainResponse.class,
-            description = "Domain ID for domain-specific OAuth provider lookup")
+            description = "Domain ID for domain-specific OAuth provider lookup", since = "4.23.0")
     private Long domainId;
 
     @Parameter(name = ApiConstants.DOMAIN, type = CommandType.STRING,
-            description = "Domain path for domain-specific OAuth provider lookup")
+            description = "Domain path for domain-specific OAuth provider lookup", since = "4.23.0")
     private String domainPath;
 
     /////////////////////////////////////////////////////

plugins/user-authenticators/oauth2/src/main/java/org/apache/cloudstack/oauth2/api/response/OauthProviderResponse.java

@@ -58,11 +58,11 @@ public class OauthProviderResponse extends BaseResponse {
     private String redirectUri;
 
     @SerializedName(ApiConstants.DOMAIN_ID)
-    @Param(description = "UUID of the domain the provider belongs to (empty for global)")
+    @Param(description = "UUID of the domain the provider belongs to (empty for global)", since = "4.23.0")
     private String domainUuid;
 
     @SerializedName(ApiConstants.DOMAIN)
-    @Param(description = "name of the domain the provider belongs to (empty for global)")
+    @Param(description = "name of the domain the provider belongs to (empty for global)", since = "4.23.0")
     private String domainName;
 
     @SerializedName(ApiConstants.ENABLED)