Skip to content

fix(contact-form): stop depending on the app autoloader in the middleware#450

Merged
YvesCesar merged 1 commit into
mainfrom
fix/contact-form-captcha-fatal
Jul 15, 2026
Merged

fix(contact-form): stop depending on the app autoloader in the middleware#450
YvesCesar merged 1 commit into
mainfrom
fix/contact-form-captcha-fatal

Conversation

@YvesCesar

Copy link
Copy Markdown
Contributor

No description provided.

…ware

Signed-off-by: YvesCesar <yvesamorim73@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://libresign.github.io/site-preview/pr-preview/pr-450/

Built to branch main at 2026-07-15 18:29 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@YvesCesar YvesCesar merged commit c11cfd2 into main Jul 15, 2026
5 checks passed
@YvesCesar YvesCesar deleted the fix/contact-form-captcha-fatal branch July 15, 2026 18:37
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Starting production deploy of site header and footer fragments...

@YvesCesar

Copy link
Copy Markdown
Contributor Author

@vitormattos

Maybe my refactoring broke the CAPTCHA:

curl -sS -H "Origin: https://libresign.coop" "https://contactform.libresign.coop/captcha.php?t=$(date +%s)"   
<br />
<b>Fatal error</b>:  Uncaught Error: Class &quot;App\Support\Http\SessionCookie&quot; not found in /var/www/html/suitecrm-form-middleware/captcha.php:10
Stack trace:
#0 {main}
  thrown in <b>/var/www/html/suitecrm-form-middleware/captcha.php</b> on line <b>10</b><br />

I reverted the change and integrated it directly into the main branch so I could test the site after deployment.

But it seems the error is still the same as in the previous PR.

Is there any additional update step beyond the GitHub pipelines needed for these changes to take effect in production?

@vitormattos

Copy link
Copy Markdown
Member

Is necessary pull the main branch at our server. I'll do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants