PHP 8 Compatibility: api

Bug #1976416 reported by Gold
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Wishlist
Gold

Bug Description

This is to track patchsets that will bring the codebase up to be compliant with PHP 8.1.

The initial run through will be on a top level directory basis. i.e. the directories below htdocs. It will focus on the results from the PHPStan and Phan static analysis tools.

This bug tracks files in the api directory.

Testing will require identifying, from the patchset, where the code is used and then verifying that that area of the site still works in both PHP 7.4 and PHP 8.1.

https://phpstan.org/
https://github.com/phan/phan

Gold (gold.catalyst)
Changed in mahara:
status: Confirmed → In Progress
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/c/mahara/+/12832
Committed: https://git.mahara.org/mahara/mahara/commit/b619397b913dbf25702aca3c462c37d30d8bbc07
Submitter: "Robert Lyon <email address hidden>"
Branch: main

commit b619397b913dbf25702aca3c462c37d30d8bbc07
Author: Gold <email address hidden>
Date: Wed Jun 1 14:37:44 2022 +1200

Bug 1976416: PHP 8 Compatibility: api

ssl_*() functions:

https://www.php.net/manual/en/function.openssl-seal.php
https://www.php.net/manual/en/function.openssl-open.php

cipher_algo (param 5) is no longer an optional parameter. The default is RC4
but is considered insecure.

Someone that knows more about what is going on here should chip in. Can we
just switch out the cipher_algo to something more reliable or does this
interact with outside sources that also need to be updated?

Change-Id: I7aa784d3bcf4fba110ab75f082ee74c6ae95eca5

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Changed in mahara:
importance: High → Wishlist
tags: added: newfeature
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.