GetPlanUrl::__invoke Argument #3 ($host) must be of type string, null given

[jasontxf]

For bug reporting only! If you're posting a feature request or discussion, please ignore.

Expected Behavior

After changing plan it should redirect back successfully using billing route.

Current Behavior

When redirecting back from the billing route, I encountered this error

Failure Information

I believe this is due to the necessary $host parameter in the new updates. But when i check the url path, host parameter dont seems to exists to begin with.

Steps to Reproduce

Please provide detailed steps for reproducing the issue.

1. Open app from the sidebar
   

2. Changing pricing plan using billing route

3. Encounter issue.

Context

Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

• Package Version: v17.3.3
• Laravel Version: v9.43.0
• PHP Version: v8.1.5

Failure Logs

Please include any relevant log snippets or files here.

Please let me know how can I further help.

[karenirenecano]

Create a query param called host and use base64_encode(Auth::user()->name) append it on the link. I had the same issue and this was my fix.

[jasontxf]

@karenirenecano Thanks for replying! Which file do I append the host exactly?

[karenirenecano]

For my case it's on the pricing page. I append it on the url like this billing/1?host=the_base_encoded_shop_name

[jasontxf]

ah! I see what you are saying now! I'll give it a try! Thanks alot for the suggestion!

Will update here again after trying.

[jasontxf]

Hello,

Here's an updates, your suggestion works very well! Thanks alot!

I created a helper function to include host in every routes to ensure host parameter is included.

function sroute($path, $params) {
    return route($path, [...$params, 'host' => base64_encode(Auth::user()->name)]);
}

That way I dont need to change my current implementation

sroute('billing', ['plan' => 2, 'shop' => Auth::user()->name]) }}"

Though @Kyon147 are we supposed to manually include host in all our links?
Thanks alot for all your time!

[Kyon147]

Hi @jasontxf

If you are using the blade templates, then yes you need to include the host for every request as you can't store it in the same way that an SPA does. As your page loads multiple times it means you need to always pass it for every page load so you can load in appbridge every time.