com_connect - form and contact mailer

Kjeld · 2022-10-31 05:40:21

peterj wrote #334052:

I had similar issues recently due to increased outgoing spam filtering by host, ended up fixing by adding the from attribute to the com_connect tag. By default I think it tries to send from the address of the person filling in the form, which may not authenticate. For best authentication the “from” and the “to” email addresses should be on the same domain as the website.

Thanks for the tip. Good to know.

All my problems vanished as soon as I had fully migrated the duits.co domain (used for the email address) to the same hosting service as dejima-tokyo.com (the domain used for the website).

jakob · 2022-11-15 14:25:04

We’ve recently encountered a problem using to_form and the most recent com_connect on PHP 8+.

Using this tag:

<txp:com_connect_select label="Department" options="{Please choose},Support,Sales" />

and the following to_form (as per the docs):

<txp:php>
    global $com_connect_form;
    switch($com_connect_form['Department'])
    {
        case 'Support':
            echo 'crew@example.com';
            break;
        case 'Sales':
            echo 'showmethemoney@example.com';
            break;
        default:
            echo 'someone@example.com';
    }
</txp:php>

the switch appears never to match, always reverting to the “default” option.

On dumping $com_connect_form in the thanks_form, it appears that $com_connect_form['Department'] is stored as an array and the switch statement cannot match it against a string. After changing the second line to:

switch($com_connect_form['Department'][0])

the to_form works again.

Have I done something wrong in the form? Or has something perhaps changed in the way it is processed in PHP8? Or does the output of com_connect_select() need to be into a string if it’s not a multiple select form?

Bloke · 2022-11-15 15:26:04

Not you. The latest version needs the [0] to work. I thought I’d updated the docs but maybe I only updated the 4.9.0 branch by mistake, or missed one of the examples. Sorry for the hassle.

jakob · 2022-11-15 15:44:54

Ah, thanks for the confirmation. No matter about the omission: good to know that my amateur code pokings were in the right direction.

BTW: this was with v 4.7.0 with my minor self-closing tags modifications.

EDIT: I submitted a PR with the update to the docs

whocarez · 2023-01-10 12:45:39

Hello,

recently I recognized, that com_connect_serverinfo stopped working in v4.7.0 with textpattern 4.8.8.

By setting something like:

<txp:com_connect_serverinfo name="REMOTE_ADDR" label="IP Nummer" />
<txp:com_connect_serverinfo name="HTTP_USER_AGENT" label="Browser" />
<txp:com_connect_serverinfo name="REQUEST_URI" label="URI" />

I get only the labels, without any data.

IP Nummer: 

Browser: 

URI:

I tested it with php 8.0.27 and php 8.1.14. So it is probably not linked to php 8.1. I also don’t see any warnings or errors in log files.

Bloke · 2023-01-10 14:11:08

Thanks for the report. I’ll run some tests later and try to find out what’s going on.

colak · 2023-01-10 17:24:46

whocarez wrote #334485:

Hello,

recently I recognized, that com_connect_serverinfo stopped working in v4.7.0 with textpattern 4.8.8.

By setting something like:

<txp:com_connect_serverinfo name="REMOTE_ADDR" label="IP Nummer" />...

I get only the labels, without any data.

IP Nummer:...

I tested it with php 8.0.27 and php 8.1.14. So it is probably not linked to php 8.1. I also don’t see any warnings or errors in log files.

Just to say that I have them working on php 8.0.12.

Bloke · 2023-01-10 18:22:46

I do suspect this is a server issue, but I can’t say for sure until I run a few tests. It may depend if the server is configured to pass that info.

Whocarez: you might get a clue from the browser inspector’s network pane. When you make requests to the server for the site, does it pass any of the server vars you’re interested in?

phiw13 · 2023-01-11 00:02:51

Bloke wrote #334489:

I do suspect this is a server issue,

Probably yes, I have the txp:com_connect_serverinfo name="REMOTE_ADDR" one working fine on different servers / hosts and different versions of PHP (8.0 and 8.1).

info.php may also tell that (search for $_SERVER['REMOTE_ADDR']).

whocarez · 2023-01-11 18:42:05

Hm, I rechecked that and with v4.6.0 it is working, but with v4.7.0 not.
phpinfo() gives normal server variables, like that

$_SERVER['HTTP_USER_AGENT']	Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:108.0) Gecko/20100101 Firefox/108.0

I fetched the plugin from github.

Bloke · 2023-01-11 18:45:31

Okay, so it’s likely to be the plugin at fault then. Thanks for checking. I’ll have to do some debugging.

whocarez · 2023-01-16 18:18:29

Had today some time for more tests.
So, if I comment out in function com_connect_serverinfo

$name = sanitizeForUrl($name);

then it works.

sanitizeForUrl makes everything lowercase, as far as I understand it. So REMOTE_ADDR gets remote_addr. And because the server variables are case sensitive, they cannot be found.

Textpattern CMS

Textpattern CMS support forum

#373 2022-10-31 05:40:21

Re: com_connect - form and contact mailer

peterj wrote #334052:

#374 2022-11-15 14:25:04

Re: com_connect - form and contact mailer

#375 2022-11-15 15:26:04

Re: com_connect - form and contact mailer

#376 2022-11-15 15:44:54

Re: com_connect - form and contact mailer

#377 2023-01-10 12:45:39

Re: com_connect - form and contact mailer

#378 2023-01-10 14:11:08

Re: com_connect - form and contact mailer

#379 2023-01-10 17:24:46

Re: com_connect - form and contact mailer

whocarez wrote #334485:

#380 2023-01-10 18:22:46

Re: com_connect - form and contact mailer

#381 2023-01-11 00:02:51

Re: com_connect - form and contact mailer

Bloke wrote #334489:

#382 2023-01-11 18:42:05

Re: com_connect - form and contact mailer

#383 2023-01-11 18:45:31

Re: com_connect - form and contact mailer

#384 2023-01-16 18:18:29

Re: com_connect - form and contact mailer

Board footer