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.

Sign up for GitHub

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

Jump to bottom

rouncube need ctype extension #7049

Closed
gwijayas opened this issue Nov 19, 2019 · 3 comments
Closed

rouncube need ctype extension #7049

gwijayas opened this issue Nov 19, 2019 · 3 comments
Labels
bug C: Documentation C: Installer
Milestone
1.4.2

Comments

@gwijayas
Copy link

gwijayas commented Nov 19, 2019 •
edited
Loading

The install requirement or installation process didn't mention/didn't check ctype extension as requirement. Some mail message will failed to read. The log is:

 PHP Fatal error:  Uncaught Error: Call to undefined function Masterminds\HTML5\Parser\ctype_alpha()

php: 7.3.11
@alecpl alecpl added this to the 1.4.2 milestone Nov 20, 2019
@alecpl
Copy link
Member

alecpl commented Nov 20, 2019

I created a ticket (Masterminds/html5-php#173) to remove that dependency. For now we'll update installer and documentation.

alecpl added a commit that referenced this issue Nov 22, 2019
@alecpl
Fix bug where Ctype extension wasn't required in Installer and INSTAL…
ffeebff 
…L file (#7049)
alecpl added a commit that referenced this issue Nov 22, 2019
@alecpl
Fix bug where Ctype extension wasn't required in Installer and INSTAL…
90bdd77 
…L file (#7049)
@alecpl
Copy link
Member

alecpl commented Nov 22, 2019

Fixed.

@alecpl alecpl closed this as completed Nov 22, 2019
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Nov 27, 2019
Add ctype dependency.
863461b 
See roundcube/roundcubemail#7049

Reported by:	Tim Bishop <tim@bishnet.net>


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@518501 35697150-7ecd-e111-bb59-0022644237b5
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Nov 27, 2019
@alexdupre
Add ctype dependency.
c831309 
See roundcube/roundcubemail#7049

Reported by:	Tim Bishop <tim@bishnet.net>
Jehops pushed a commit to Jehops/freebsd-ports-legacy that referenced this issue Nov 27, 2019
@alexdupre
Add ctype dependency.
44fcafc 
See roundcube/roundcubemail#7049

Reported by:	Tim Bishop <tim@bishnet.net>


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@518501 35697150-7ecd-e111-bb59-0022644237b5
nextgens added a commit to nextgens/Mailu that referenced this issue Nov 25, 2022
@nextgens
Turns out that php81-ctype is required by roundcube
e927426 
see roundcube/roundcubemail#7049
@nextgens nextgens mentioned this issue Nov 25, 2022
Merged
@nextgens
Copy link

FYI https://github.com/roundcube/roundcubemail/wiki/Install-Requirements still doesn't mention ctype

bors bot added a commit to Mailu/Mailu that referenced this issue Nov 25, 2022
@bors @nextgens
Merge #2542 #2559
033889d 
2542: Implement header authentication via external proxy r=mergify[bot] a=nextgens

## What type of PR?

Feature

## What does this PR do?

Implement header authentication via external proxy

### Related issue(s)
- closes #1972
- closes #2183

## Prerequisites
Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

- [x] In case of feature or enhancement: documentation updated accordingly
- [x] Unless it's docs or a minor change: add [changelog](https://mailu.io/master/contributors/workflow.html#changelog) entry file.


2559: Turns out that php81-ctype is required by roundcube r=mergify[bot] a=nextgens

## What type of PR?

bug-fix

## What does this PR do?

It solves:
```
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "NOTICE: PHP message: PHP Fatal error:  Uncaught Error: Call to undefined function Masterminds\HTML5\Parser\ctype_alpha() in /var/www/roundcube/vendor/masterminds/html5/src/HTML5/Parser/Tokenizer.php:140"
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "Stack trace:"
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "#0 /var/www/roundcube/vendor/masterminds/html5/src/HTML5/Parser/Tokenizer.php(82): Masterminds\HTML5\Parser\Tokenizer->consumeData()"
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "#1 /var/www/roundcube/vendor/masterminds/html5/src/HTML5.php(161): Masterminds\HTML5\Parser\Tokenizer->parse()"
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "#2 /var/www/roundcube/vendor/masterminds/html5/src/HTML5.php(89): Masterminds\HTML5->parse('<html>\n    <hea...', Array)"
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "#3 /var/www/roundcube/program/lib/Roundcube/rcube_washtml.php(700): Masterminds\HTML5->loadHTML('<html>\n    <hea...')"
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "#4 /var/www/roundcube/program/actions/mail/index.php(975): rcube_washtml->wash('<html>\n    <hea...')"
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "#5 /var/www/roundcube/program/actions/mail/index.php(1019): rcmail_action_mail_index::wash_html('<!doctype html>...', Array, Array)"
[25-Nov-2022 08:19:20] WARNING: [pool php] child 335 said into stderr: "#6 /var/www/roundcube/program/actions/mail/show.php(720): rcmail_action_mail_index::pr..."
```

see roundcube/roundcubemail#7049


Co-authored-by: Florent Daigniere <nextgens@freenetproject.org>
svmhdvn pushed a commit to svmhdvn/freebsd-ports that referenced this issue Jan 10, 2024
@alexdupre
Add ctype dependency.
cc609a0 
See roundcube/roundcubemail#7049

Reported by:	Tim Bishop <tim@bishnet.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug C: Documentation C: Installer
Projects
None yet
Milestone
1.4.2
Development

No branches or pull requests
3 participants
@nextgens @alecpl @gwijayas