View Issue Details

IDProjectCategoryView StatusLast Update
0030794mantisbtsignuppublic2023-02-22 19:21
Reporterdregad Assigned Todregad  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Target Version2.25.6Fixed in Version2.25.6 
Summary0030794: Captcha image not showing on PHP 8.1
Description

Securimage 3.6.8 is not compatible with PHP 8.1. The catpcha image is not displayed on signup_page.php (see attached screenshot)

Loading the image directly in a browser tab shows a very large number of PHP Deprecated notices, on various line numbers:

Implicit conversion from float 5.375 to int loses precision in /path/to/securimage/securimage.php on line 2254

The problem has been reported upstream and a pull request has been provided (https://github.com/dapphp/securimage/pull/112) but the library's author is no longer maintaining it, so it is highly unlikely that the PR will be merged and a new release cut.

Until we have the resources to replace Securimage (see 0027156), we should patch it manually in our fork of the repository https://github.com/mantisbt/securimage which is already the source for Composer.

TagsNo tags attached.
Attached Files
image.png (16,090 bytes)   
image.png (16,090 bytes)   

Relationships

related to 0027156 confirmed Replace SecurImage Captcha library 
related to 0030814 closeddregad Captcha audio not working 

Activities

dregad

dregad

2022-07-30 10:51

developer   ~0066838

The 'mantis' branch of our Securimage fork has been updated to fix the PHP 8.1 compatibility issues.

https://github.com/mantisbt/securimage/compare/01d5860a023382b4076cb98e143828357e679874...17a390243d831017f9ce9ee23c9dc481e7554ea0

Changes will be pulled in via composer update.

dregad

dregad

2022-07-30 11:44

developer   ~0066842

PR https://github.com/mantisbt/mantisbt/pull/1836

Related Changesets

MantisBT: master-2.25 5d5d74f0

2022-07-30 11:17

dregad


Details Diff
Update Securimage library to custom 3.6.8 version

The 'mantis' branch of our Securimage fork [1] was updated to fix
PHP 8.1 compatibility issues. The initial patch was submitted as a PR to
the upstream repository [2], and a few additional fixes covering the
audio-related scripts were committed and submitted to the PR's author.

This commit updates the composer lock file, to pull these changes in.

Fixes 0030794

[1]: https://github.com/mantisbt/securimage
[2]: https://github.com/dapphp/securimage/pull/112
Affected Issues
0030794
mod - composer.lock Diff File