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.

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

Add PHP CURL extension check to the first installation step #5145

Closed
robwent opened this issue Oct 13, 2017 · 9 comments

Comments

@robwent
Copy link
Contributor

commented Oct 13, 2017

This is based on 2 reports from slack about issues with the dashboard on first load.

On the first load the dashboard shows a 500 error but the left menu loads.

Refreshing the browser loads the dashboard without issues.

Q A
Bug report? X
Feature request?
Enhancement? X

Description:

People starting a server with ubuntu don't have php-curl installed by default and the dashboard can show a 500 error on the first load but be fine when reloading.

@escopecz found the problem/fix here https://mautic.slack.com/archives/C02HV76BW/p1507820618000842

The install script that checks php extensions doesn't check for curl that causes the error.

@escopecz escopecz changed the title 500 Error on first dashboard load Add PHP CURL extension check to the first installation step Oct 13, 2017
@escopecz escopecz added the Bug label Oct 13, 2017
@YosuCadilla

This comment has been minimized.

Copy link

commented Oct 13, 2017

I was one of the 2 people having this problem, just installing the php-curl package solved the issue.

No idea if it's important but I had curl installed on the server, but not the php-curl package, maybe that is why the pre-install check didn't think it was missing?

This was my setup in case it can be of help: apache 2.4.xx / php 7.0.xx / mariadb 10 / ubuntu 16.04.xx

@Maxell92

This comment has been minimized.

Copy link
Contributor

commented Oct 13, 2017

It seems that Mautic does not check for curl PHP extension.

@heathdutton heathdutton modified the milestone: 2.14.1 Sep 4, 2018
@heathdutton

This comment has been minimized.

Copy link
Member

commented Sep 5, 2018

Related: #4390

While adding composer dependency checking is great, these issues request the installer does a check as well. Need PRs.

@npracht npracht added this to the 2.14.2 milestone Sep 5, 2018
@escopecz escopecz removed this from the 2.14.2 milestone Oct 16, 2018
@Aerendir

This comment has been minimized.

Copy link
Contributor

commented Jun 29, 2019

I was opening an issue for the exact same problem, then searched for an already opened one and came to this.

A check about cURL is really helpful: I fully configured the server following the directives of the installation step, but it didn't alert me about cURL.

The result is that after login, I get a 500 error on the dashboard.

The logs all reference cURL constants that doesn't exist.

@Aerendir

This comment has been minimized.

Copy link
Contributor

commented Jun 29, 2019

Which branch should have I to use to submit a PR?

@escopecz

This comment has been minimized.

Copy link
Member

commented Jun 29, 2019

Staging. The default one.

@Aerendir

This comment has been minimized.

Copy link
Contributor

commented Jun 29, 2019

I will submit a new PR using the default staging branch.

@Aerendir

This comment has been minimized.

Copy link
Contributor

commented Jun 29, 2019

@escopecz , here the new PR against the staging branch: #7657

@Woeler Woeler closed this in 91109fa Aug 17, 2019
Woeler added a commit that referenced this issue Aug 17, 2019
Fix #5145: Check that `cURL` is installed.
@tmdpny

This comment has been minimized.

Copy link

commented Oct 12, 2019

I was one of the 2 people having this problem, just installing the php-curl package solved the issue.

No idea if it's important but I had curl installed on the server, but not the php-curl package, maybe that is why the pre-install check didn't think it was missing?

This was my setup in case it can be of help: apache 2.4.xx / php 7.0.xx / mariadb 10 / ubuntu 16.04.xx

I just want to say this WAS important. I've been pulling my hair out this morning on this - curl is installed but not detected... but tried php-curl and voila - fixed. THANK YOU

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
8 participants
You can’t perform that action at this time.