Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: protected responseContentType to allow overloading of fetchUrl function #446

Conversation

ricklambrechts
Copy link
Contributor

Also set responseContentType to protected to allow proper overloading of fetchUrl.

Related: #433

@DeepDiver1975 DeepDiver1975 merged commit f7c91b9 into jumbojett:master Sep 27, 2024
9 checks passed
@ricklambrechts ricklambrechts deleted the fix-override-response-content-type branch September 27, 2024 11:02
Magentron added a commit to Magentron/OpenID-Connect-PHP that referenced this pull request Nov 7, 2024
* upstream/master:
  fix: protected responseContentType to allow overloading of fetchUrl function (jumbojett#446)
  test: unit tests for verifyJWTClaims and different aud claims (jumbojett#443)
  Fix TypeError in `verifyJWTClaims` (jumbojett#442)
  release: v1.0.2 (jumbojett#439)
  test: add unit test for SERVER_PORT type cast (jumbojett#438)
  fix: bring back jumbojett#404 (jumbojett#437)
  release: v1.0.1 (jumbojett#432)
  fix: protected $responseCode to allow proper overloading of fetchURL() (jumbojett#433)
  chore(deps-dev): update yoast/phpunit-polyfills requirement from ^1.0 to ^2.0 (jumbojett#430)
  chore(deps): update phpseclib/phpseclib requirement from ~3.0 to ^3.0.7
  ci: run GitHub workflows on pull requests and pushes to master (jumbojett#431)
  chore: enable dependabot for composer (jumbojett#429)
  fix: handle JWT decode of non JWT tokens (jumbojett#428)
  fix: method signatures after 1.0 release (jumbojett#427)
Magentron added a commit to Magentron/OpenID-Connect-PHP that referenced this pull request Nov 7, 2024
…est-token

* master:
  fix: protected responseContentType to allow overloading of fetchUrl function (jumbojett#446)
  test: unit tests for verifyJWTClaims and different aud claims (jumbojett#443)
  Fix TypeError in `verifyJWTClaims` (jumbojett#442)
  release: v1.0.2 (jumbojett#439)
  test: add unit test for SERVER_PORT type cast (jumbojett#438)
  fix: bring back jumbojett#404 (jumbojett#437)
  release: v1.0.1 (jumbojett#432)
  fix: protected $responseCode to allow proper overloading of fetchURL() (jumbojett#433)
  chore(deps-dev): update yoast/phpunit-polyfills requirement from ^1.0 to ^2.0 (jumbojett#430)
  chore(deps): update phpseclib/phpseclib requirement from ~3.0 to ^3.0.7
  ci: run GitHub workflows on pull requests and pushes to master (jumbojett#431)
  chore: enable dependabot for composer (jumbojett#429)
  fix: handle JWT decode of non JWT tokens (jumbojett#428)
  fix: method signatures after 1.0 release (jumbojett#427)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants