Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Carbtest patch 1#445

Open
carbtest wants to merge 4 commits intoapereo:master from
carbtest:carbtest-patch-1
Open

Carbtest patch 1 #445
carbtest wants to merge 4 commits intoapereo:master from
carbtest:carbtest-patch-1

Conversation

@carbtest
Copy link
Contributor

@carbtest carbtest commented Feb 13, 2025

No description provided.

Deprecated: phpCAS::client(): Implicitly marking parameter $sessionHandler as nullable is deprecated, the explicit nullable type must be used instead
Copy link
Contributor Author

carbtest commented Feb 13, 2025
edited
Loading

These changes will fix: PHP8.4 issues
Deprecated: phpCAS::client(): Implicitly marking parameter $sessionHandler as nullable is deprecated, the explicit nullable type must be used instead in /vendor/apereo/phpcas/source/CAS.php on line 348

Deprecated: phpCAS::proxy(): Implicitly marking parameter $sessionHandler as nullable is deprecated, the explicit nullable type must be used instead in /vendor/apereo/phpcas/source/CAS.php on line 403

Deprecated: CAS_Client::__construct(): Implicitly marking parameter $sessionHandler as nullable is deprecated, the explicit nullable type must be used instead in /vendor/apereo/phpcas/source/CAS/Client.php on line 933

Copy link

Really hope someone who has the ability to integrate this change will do so. Tested and confirmed it works with PHP 8.4. The changes are so simple...please someone release a phpCAS 1.6.2 version with these simple changes for PHP 8.4 support. It's been 3 months since PHP 8.4 was officially released and we all need this updated version with PHP 8.4 support.

floverdevel reacted with thumbs up emoji

Copy link
Contributor Author

carbtest commented Mar 5, 2025

Really hope someone who has the ability to integrate this change will do so. Tested and confirmed it works with PHP 8.4. The changes are so simple...please someone release a phpCAS 1.6.2 version with these simple changes for PHP 8.4 support. It's been 3 months since PHP 8.4 was officially released and we all need this updated version with PHP 8.4 support.

This repo is no longer maintained, we will have to fork it or modify our local version ;(

Copy link

rutgers-master commented Mar 6, 2025
edited by unfurl-links bot
Loading

From what I read here: https://groups.google.com/a/apereo.org/g/cas-user/c/7A4MKl6DIOk/m/WikoKAJ2AgAJ
They are still willing to review PRs... Fingers crossed someone does....

Copy link

We're going to try this after talking with the original developer of phpCAS who unfortunately stepped down and gave back all his privileges last year...

@phy25 Can you help us get this PR approved? Joachim said that you may still have commit privileges. You would make lots of people very happy if you could get this very simple change approved. Thank you!

Copy link
Contributor

adoy commented Mar 19, 2025

+1 for this. One of my client is upgrading to PHP8.4 and would profit from this change. I can help to merge / release if needed. Not sure I have time to improve all the code but at least I could maintain compatibility with PHP versions.

floverdevel reacted with thumbs up emoji

Copy link
Contributor Author

carbtest commented Mar 19, 2025 via email

Much Appreciated From: Pierrick Charron ***@***.***> Sent: Wednesday, March 19, 2025 11:44 AM To: apereo/phpCAS ***@***.***> Cc: Web Master ***@***.***>; Author ***@***.***> Subject: Re: [apereo/phpCAS] Carbtest patch 1 (PR #445) +1 for this. One of my client is upgrading to PHP8.4 and would profit from this change. I can help to merge / release if needed. Not sure I have time to improve all the code but at least I could maintain compatibility with PHP versions. — Reply to this email directly, view it on GitHub<#445 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/A6CG2TUS4YZLL4RQDFLVNLD2VG3GFAVCNFSM6AAAAABXDCICWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMZXGY4DAMRZGI>. You are receiving this because you authored the thread.Message ID: ***@***.******@***.***>> [adoy]adoy left a comment (apereo/phpCAS#445)<#445 (comment)> +1 for this. One of my client is upgrading to PHP8.4 and would profit from this change. I can help to merge / release if needed. Not sure I have time to improve all the code but at least I could maintain compatibility with PHP versions. — Reply to this email directly, view it on GitHub<#445 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/A6CG2TUS4YZLL4RQDFLVNLD2VG3GFAVCNFSM6AAAAABXDCICWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMZXGY4DAMRZGI>. You are receiving this because you authored the thread.Message ID: ***@***.******@***.***>>
floverdevel reacted with thumbs up emoji

Copy link

rutgers-master commented Mar 21, 2025
edited by unfurl-links bot
Loading

Here's from the CAS primary maintainer after reaching out to see if he could help:

I am in no position and possess no relevant skill to review, comment, accept or merge changes, and certainly in no position to release the project. Furthermore, I have no time to spend on these tasks, and at the moment it seems to me that we don't really have a designated replacement in the community with a good enough proven reputation and needed time commitment to carry the torch going forward and be granted access.

I don't quite know the protocol is here; all I can suggest atm is, you can fork the repository and treat it just as any other open source project that is unsustained, not funded and therefore unmaintained and abandoned/stable. You can reach out to the Apereo foundation and discuss this with them. If their decision is that person X is to be given the rights to maintain the project going forward, I can help with that or Apereo itself can.

He maintains CAS itself. phpCAS is not his project and he's never done anything with this project, so I don't fault him in any way and appreciate that he took the time to reply to my email. If someone wants to reach out to Apereo themselves and try to get rights to maintain this phpCAS project going forward, go for it.

Good luck to anyone with more skills that can continue this project, hate to see it go away, but I'm at the extent of my abilities, so I hope someone reaches out to the Apereo group (https://www.apereo.org/contact) and can save this project.

timmy78 added a commit to timmy78/phpCAS that referenced this pull request Jul 2, 2025
timmy78 added a commit to timmy78/phpCAS that referenced this pull request Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Comments

AltStyle によって変換されたページ (->オリジナル) /