Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strip port from IP address #36

Open
kainula opened this issue Jun 22, 2022 · 12 comments
Open

Strip port from IP address #36

kainula opened this issue Jun 22, 2022 · 12 comments

Comments

@kainula
Copy link

kainula commented Jun 22, 2022

Facebook conversions API does not accept IP addresses with port number. Parameter ip_override, which is used as a user IP address, can have a port.

Modify the ip_override so that the port is removed.

@SebbeJohansson
Copy link

I have the same question for Meta. I am trying to talk to them about this, but currently the only solution is to make our own version of the template.

@Bukashk0zzz
Copy link

Bukashk0zzz commented Mar 3, 2023

@SebbeJohansson @kainula Why do you not use the Stape version of the template?
It supports all functionality that the official tag has and adds on top of its additional features.

We maintain our templates and fix any issues fast.

@SebbeJohansson
Copy link

@Bukashk0zzz Hi! I am in discussion with Meta about the stape template (haven't been in contact with anyone from stape so haven't been able to go directly there).
The stape io template has a much worse performance compared to the official Meta one. Several features in the stape io template is useless for us, but that cause worse performance (like pinging back to the session to create a cookie). We had an issue with collect events both to GA4 and FB timing out, which turned out to be because the stape io template didn't process fast enough.

Currently our solution for our issues (both this and sending email and phone number with our ga4 mapping of data) was to modify the official template. It could be that we are missing something with how to use the Stape template. Not sure!

Maybe, this is because of the amount of data we are processing. Not sure!

If you or anyone else from Stape want to talk to me about this, feel free to reach out to my email at [email protected]

@Bukashk0zzz
Copy link

@SebbeJohansson
You were the first one who mentioned performance issues inside our template. That's a pretty interesting case.
We have customers with different amounts of traffic and never had such an issue.
The official template works with cookies in the same way as Stape because there are no other methods to do this inside sGTM.

I believe that code of the Stape template is processing a little bit longer than the Official one, but this is like 2 ms with 3 ms. Almost all the time, tags wait for responses from the platform.

The Stape template uses a new CAPI version. Maybe that's why you have a longer response time. Or maybe you need more sGTM server instances to handle your traffic.

@SebbeJohansson
Copy link

@Bukashk0zzz interesting! The official one doesn't send any call back to the client at all for us. Maybe I'm checking wrong.

We went from having requests missing from timing out to everything working fine. Weird that no one else is reporting issues.

We are at the same number of servers so it's not that. Is there someone that I can email to talk about this directly with stape? The stape one has features that we need (remapping user data among others) so I had to modify (aka borrow code from your one) to add into the official Meta one. We would love to use the stape one instead, but currently it's not an option.

@koconnor3
Copy link

koconnor3 commented Mar 3, 2023 via email

@SebbeJohansson
Copy link

@koconnor3 so the only reason the official one doesn't completely crash our sgtm servers are because its not up to date with the repo. That is kinda hilarous.

@koconnor3
Copy link

koconnor3 commented Mar 3, 2023 via email

@SebbeJohansson
Copy link

@koconnor3 the call going from sgtm back to the client was timing out and in turn, the collection events going both to FB and ga4 was timing out. I dont know why (still waiting for Google to finish their internal investigation), but it was happening. Switching to the official one (with some modding from me) worked.

I did see that StapeIO has an "offline" template (not sure what the difference is) but didn't look like it was doing the problematic stuff.

@Bukashk0zzz
Copy link

If this is still actual @SebbeJohansson can you submit a request here with a detailed description https://help.stape.io/hc/en-us ?
Please refer to this issue and please mention that Denis asked you to create this ticket. We at Stape will try to reproduce it.

1 similar comment
@Bukashk0zzz
Copy link

If this is still actual @SebbeJohansson can you submit a request here with a detailed description https://help.stape.io/hc/en-us ?
Please refer to this issue and please mention that Denis asked you to create this ticket. We at Stape will try to reproduce it.

@SebbeJohansson
Copy link

Hi @Bukashk0zzz. It is still present. I'll submit a ticket!

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

No branches or pull requests

4 participants