permalink never set (or used)

[ehoch]

I noticed the readme makes reference to sending the permalink:

permalink     : the permanent URL for the entry the comment belongs to

But then it's never used in the actual code.

[ehoch]

@joshfrench Just pinging you on this. Is the app still maintained? Should I do a pull request fixing this?

[joshfrench]

Sorry for the delay! Akismet does collect that parameter, and omitting from Rakismet was an oversight. I'd love a pull request if you feel like implementing it. Thanks for catching that!

[ehoch]

Yeah it seems as simple as adding it to a few arrays of yours. I'll gladly submit a PR.

I've been having MASSIVE spam issues on my sites and want to see any additional data points I can give akismet. Are there any other parameters they take? Where did you find the specs when creating this?

All I can find is parsing through library code, where I saw they take the PHP _SERVER array. Curious if they would take some kind of RoR equivalent or any other data points we're missing..

[joshfrench]

The official API docs are here: http://akismet.com/development/api/

It's not clear if they recommend sending $_SERVER in addition to the listed params, or if they're just suggesting you use that array to populate the data you send back to them. None of their code samples send $_SERVER whole, so I'd always assumed it was the latter.

They've been pretty helpful in the past though, it would be a good question for them.

[ehoch]

@joshfrench Looks like they will accept any HTTP header as a parameter to help them parse. Exact wording from them:

Some of it would certainly help.  The PHP $_SERVER array contains CGI variables and HTTP headers roughly according to the CGI spec: http://www.faqs.org/rfcs/rfc3875.html

In particular it's the HTTP_ headers such as HTTP_ACCEPT, HTTP_VIA, HTTP_FORWARDED_FOR and so on that are especially useful.  If you can pass them named in the CGI fashion (uppercase with underscores) then we'll be able to use them immediately.

[ehoch]

@joshfrench I've submitted two PRs to you. One with the permalink fix. The other will send HTTP_ headers (all they care about from $_SERVER -- if middleware is enabled). All of this is from communication with them.

Let me know if you can merge next week and I can work with Akismet to make sure they're using these data points from us.

[ehoch]

Closed via 17c9e1c