Deprecate not passing server base url and secret (#195)

* Deprecate not passing server base url and secret

* Update src/BigBlueButton.php

Co-authored-by: Felix Jacobi <felix@jacobi-hamburg.net>

* Update src/BigBlueButton.php

Co-authored-by: Felix Jacobi <felix@jacobi-hamburg.net>

* Show deprecation when using environment variables

---------

Co-authored-by: Felix Jacobi <felix@jacobi-hamburg.net>

Author: samuelwei

src/BigBlueButton.php

@@ -116,6 +116,22 @@ class BigBlueButton
      */
     public function __construct(?string $baseUrl = null, ?string $secret = null, ?TransportInterface $transport = null, string $hashingAlgorithm = HashingAlgorithm::SHA_1)
     {
+        if (null === $baseUrl) {
+            @trigger_error(sprintf('Constructing "%s" without passing a server base URL is deprecated and will throw an exception 6.0.', self::class), \E_USER_DEPRECATED);
+        }
+
+        if (null === $secret) {
+            @trigger_error(sprintf('Constructing "%s" without passing a secret is deprecated and will throw an exception 6.0.', self::class), \E_USER_DEPRECATED);
+        }
+
+        if (getenv('BBB_SECURITY_SALT') !== false || getenv('BBB_SECRET') !== false) {
+            @trigger_error('Using BBB_SECURITY_SALT or BBB_SECRET environment variables is deprecated for security reasons and will be removed in 6.0. Use the constructor parameters instead.', \E_USER_DEPRECATED);
+        }
+
+        if (getenv('BBB_SERVER_BASE_URL') !== false) {
+            @trigger_error('Using BBB_SERVER_BASE_URL environment variable is deprecated for security reasons and will be removed in 6.0. Use the constructor parameters instead.', \E_USER_DEPRECATED);
+        }
+
         // Keeping backward compatibility with older deployed versions
         $this->securitySecret = $secret ?: getenv('BBB_SECURITY_SALT') ?: getenv('BBB_SECRET');
         $this->bbbServerBaseUrl = $baseUrl ?: getenv('BBB_SERVER_BASE_URL');