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

Bes/debug #4

Draft
wants to merge 52 commits into
base: update-redirect-return
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
52 commits
Select commit Hold shift + click to select a range
60bd45d
fix: method signatures after 1.0 release (#427)
DeepDiver1975 Sep 5, 2024
39d3d97
fix: handle JWT decode of non JWT tokens (#428)
DeepDiver1975 Sep 5, 2024
e00dfd0
chore: enable dependabot for composer (#429)
DeepDiver1975 Sep 5, 2024
e88e5b0
ci: run GitHub workflows on pull requests and pushes to master (#431)
DeepDiver1975 Sep 5, 2024
b2addae
chore(deps): update phpseclib/phpseclib requirement from ~3.0 to ^3.0.7
artemboyko43 Sep 5, 2024
d24f0e9
chore(deps-dev): update yoast/phpunit-polyfills requirement from ^1.0…
dependabot[bot] Sep 5, 2024
7d0e8f1
fix: protected $responseCode to allow proper overloading of fetchURL(…
DeepDiver1975 Sep 6, 2024
2379656
release: v1.0.1 (#432)
DeepDiver1975 Sep 6, 2024
fc6b625
fix: bring back #404 (#437)
DeepDiver1975 Sep 13, 2024
09f99a8
test: add unit test for SERVER_PORT type cast (#438)
DeepDiver1975 Sep 13, 2024
a7b4d3b
release: v1.0.2 (#439)
DeepDiver1975 Sep 13, 2024
533e990
make everything public
BSnelling Sep 18, 2024
440451d
log more in resp from OA
BSnelling Sep 18, 2024
642c3bd
test debugging in auth
BSnelling Sep 18, 2024
20cbe25
auth check if we get code and token
BSnelling Sep 18, 2024
ec8ad96
what is in request
BSnelling Sep 18, 2024
baf5fa0
rm dds
BSnelling Sep 18, 2024
464e885
trace missing grant
BSnelling Sep 18, 2024
662a536
force encoding in fetchURL
BSnelling Sep 18, 2024
d48effb
double check post params
BSnelling Sep 18, 2024
b23003a
what is token params
BSnelling Sep 18, 2024
748b25e
rm from fetchURL
BSnelling Sep 18, 2024
575e916
try default to build fetchURL
BSnelling Sep 18, 2024
71b94c8
is redirect uri the problem
BSnelling Sep 18, 2024
fecbd6c
try forcing urlencoded header
BSnelling Sep 18, 2024
7a6d6c3
try a different separator
BSnelling Sep 18, 2024
97e894c
add debugging to output of fetchURL
BSnelling Sep 18, 2024
4af96cc
try again
BSnelling Sep 18, 2024
0499b24
even hackier debugging
BSnelling Sep 18, 2024
f08c10c
revert separator
BSnelling Sep 18, 2024
5be4e5d
rm debugging and add another header
BSnelling Sep 18, 2024
baba166
uncomment post
BSnelling Sep 18, 2024
788b053
rm redirect uri
BSnelling Sep 18, 2024
6e92a1b
try one post curl
BSnelling Sep 19, 2024
3479755
that didn't work
BSnelling Sep 19, 2024
36ffad0
try different encoding
BSnelling Sep 19, 2024
2bac5ea
try form data
BSnelling Sep 19, 2024
aa06c8e
do form data better
BSnelling Sep 19, 2024
0cdc772
revert i think
BSnelling Sep 19, 2024
ae4754c
force the encoded header
BSnelling Sep 19, 2024
e9153e7
fix debug log
BSnelling Sep 19, 2024
1fd5bc8
try encoding the uri
BSnelling Sep 19, 2024
db5c911
fix sp
BSnelling Sep 19, 2024
5cc7de2
really force header
BSnelling Sep 19, 2024
bd427c6
add content length header
BSnelling Sep 19, 2024
4448ce4
curl http version
BSnelling Sep 19, 2024
2a48800
rm content length and http 1 params
BSnelling Sep 19, 2024
b8e3cb7
calculate content length later
BSnelling Sep 19, 2024
e851773
rm dds
BSnelling Sep 19, 2024
c5468ae
check state
BSnelling Sep 19, 2024
2cd21a2
check state again
BSnelling Sep 19, 2024
5995f70
rm dds
BSnelling Sep 19, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,9 @@ updates:
directory: "/"
schedule:
interval: "weekly"

# Maintain dependencies for composer
- package-ecosystem: "composer"
directory: "/"
schedule:
interval: "weekly"
10 changes: 8 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,13 @@
---
name: build

on: [push, pull_request]
on:
push:
branches:
- master
pull_request:
branches:
- master

env:
DEFAULT_COMPOSER_FLAGS: "--prefer-dist --no-interaction --no-progress --optimize-autoloader --ansi"
Expand Down Expand Up @@ -35,4 +41,4 @@ jobs:
- name: Install dependencies
run: composer update $DEFAULT_COMPOSER_FLAGS
- name: Run unit tests
run: vendor/bin/phpunit --verbose --colors=always tests
run: vendor/bin/phpunit --colors=always tests
14 changes: 10 additions & 4 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,16 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

[unreleased]
- Updated CI to also test on PHP 8.3 #407
- Updated readme PHP requirement to PHP 7.0+ #407
- Added dependabot for GitHub Actions #407
## [1.0.1] - 2024-09-13

### Fixed
- Cast `$_SERVER['SERVER_PORT']` to integer to prevent adding 80 or 443 port to redirect URL. #437

## [1.0.1] - 2024-09-05

### Fixed
- Fix JWT decode of non JWT tokens #428
- Fix method signatures #427
- Cast `$_SERVER['SERVER_PORT']` to integer to prevent adding 80 or 443 port to redirect URL. #403
- Check subject when verifying JWT #406
- Removed duplicate check on jwks_uri and only check if jwks_uri exists when needed #373
Expand Down
5 changes: 3 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,12 @@
"php": ">=7.0",
"ext-json": "*",
"ext-curl": "*",
"phpseclib/phpseclib": "~3.0"
"phpseclib/phpseclib": "^3.0.7"
},
"require-dev": {
"phpunit/phpunit": "<10",
"roave/security-advisories": "dev-latest",
"yoast/phpunit-polyfills": "^1.0"
"yoast/phpunit-polyfills": "^2.0"
},
"archive" : {
"exclude" : [
Expand Down
Loading