9 maybe security bug getting ip address

[lukasmatusiewicz]

This pull request includes several changes to the PrivacyIDEA client and its related provider. The most important changes include modifications to how the client IP is forwarded and improvements to the PrivacyIDEA client instantiation.

PrivacyIDEA client changes:

• lib/PIClient/PrivacyIDEA.php: Changed the type of the forwardClientIP property from bool to string and updated the logic to forward the client IP if it is not empty. [1] [2] [3]

PrivacyIDEA provider changes:

• lib/Provider/PrivacyIDEAProvider.php: Updated the createPrivacyIDEAInstance method to set the forwardClientIP property only if the client IP is available.
• lib/Provider/PrivacyIDEAProvider.php: Modified the getClientIP method to return a string and log an error if the client IP cannot be retrieved.

This pull request also includes several changes primarily focused on simplifying the codebase and removing redundant configurations. The most important changes include the removal of ESLint and Stylelint workflows, updates to the dependabot.yml configuration, and various code style improvements.

Workflow and Configuration Updates:

• .github/workflows/lint-eslint.yml: Removed the ESLint workflow configuration.
• .github/workflows/lint-stylelint.yml: Removed the Stylelint workflow configuration.
• .github/dependabot.yml: Removed redundant Composer package ecosystems from the Dependabot configuration.

Code Style Improvements:

• lib/PIClient/PIChallenge.php: Converted double quotes to single quotes for string literals.
• lib/PIClient/AuthenticationStatus.php: Converted constants to use single quotes and public visibility.

These changes help streamline the codebase and remove unnecessary configurations, making the project easier to maintain.

[nilsbehlen]

if you are fine with the lint actions not running properly, you can merge this. the code looks fine.

[lukasmatusiewicz]

the worflows are already fixed, but the updated version will be executed first at the next pr