Minor refactoring

[darshanasbg]

Purpose

This pull request includes a variety of changes aimed at improving code readability, fixing inconsistencies, and removing unused functionality. The most important changes include renaming methods and variables for clarity, replacing shorthand variable declarations with explicit ones, removing unused functions, and handling potential errors in JSON encoding.

Code readability improvements:

• Renamed initMultiPlexer to initMultiplexer to fix a typo and ensure consistency in naming throughout the codebase. (cmd/server/main.go) [1] [2]
• Updated log message in startServer to improve clarity by changing "Starting Asgardeo Thunder..." to "Asgardeo Thunder started...". (cmd/server/main.go)
• Corrected capitalization of "client Id" to "client id" in multiple comments for consistency. (internal/application/service/applicationservice.go, internal/identity/oauth2/token/tokenhandler.go) [1] [2]

Code simplification:

• Replaced shorthand variable declarations (e.g., redirectURIs := []string{}) with explicit declarations (e.g., var redirectURIs []string) for better readability. (internal/application/service/applicationservice.go, internal/system/database/client/dbclient.go) [1] [2] [3]

Removal of unused functionality:

• Removed the unused IsValidRedirectURI function, which checked the validity of redirect URIs for OAuth applications. (internal/application/utils/utils.go)

Error handling:

• Added error handling for the JSON encoding process in WriteJSONError to ensure the function exits gracefully if encoding fails. (internal/utils/httputil.go)

Documentation updates:

• Updated the comment for the ServiceInterface struct to clarify its purpose in handling routes. (internal/services/service.go)