Merge Main with feature/71 (#81)

* Possible login fix (#72)

Change checks to empty() and movement of try statements

* Adding statement brackets (#73)

* 🔒 Force Single Logout (#77)

Signed-off-by: Luke Tainton <luke@tainton.uk>

* Update assigned-issues-inprogress.yml

Co-authored-by: Luke Tainton <luke@tainton.uk>
This commit was merged in pull request #81.
This commit is contained in:
Alexander Davis
2020-08-13 20:18:35 +01:00
committed by GitHub
parent e2c5bcf24b
commit d5c8ae1c23
3 changed files with 6 additions and 5 deletions

View File

@@ -10,6 +10,6 @@ jobs:
steps:
- uses: alex-page/github-project-automation-plus@v0.2.4
with:
project: FHeD v1.0
column: In Progress
repo-token: ${{ secrets.AUTO_TOKEN }}
project: v1.0
column: In progress
repo-token: ${{ secrets.GITHUB_TOKEN }}

View File

@@ -5,6 +5,7 @@
// Perform the OIDC authentication
try {
$oidc->authenticate();
$_SESSION['access_token'] = $oidc->requestClientCredentialsToken()->access_token;
$oidc_user = array(
'sub' => $oidc->requestUserInfo('sub'),
'username' => $oidc->requestUserInfo('preferred_username'),

View File

@@ -2,7 +2,7 @@
$PAGE_NAME = "Logging out...";
require_once __DIR__ . "/../../includes/prereqs.php";
$access_token = $_SESSION['access_token'];
session_destroy();
header('Location: /');
$oidc->signOut($access_token, $_ENV['APP_URL']);
?>