I confirm cre3at1v3.
In fact, in that point of code where you injected the shortlink, the satoshis were actually already sent. From a faucet user POV, it means nothing to show a shortlink as long as satoshis were already received.

One user can just close the window and not even solve that shortlink.

The treatment needs to be done before the line « $result = $faucethub->send($addy, $amount, "false", $userip); » any way.

The header function is technically just cosmetic, in order to confirm that the satoshis were successfully sent, even though they already succeeded before the shortlink.