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

Calculator doesn't work for 22+ services #172

Open
46bit opened this issue Apr 25, 2019 · 7 comments
Open

Calculator doesn't work for 22+ services #172

46bit opened this issue Apr 25, 2019 · 7 comments

Comments

@46bit
Copy link
Contributor

46bit commented Apr 25, 2019

The pricing calculator passes around the service data in its GET query string. This hits haproxy's size limit. This bug was already recorded on paas-billing, but fixing it would require changes to both.

Ideally a fix would allow all these things:

  • Pricing calculator links are still sharable;
  • Pricing calculator doesn't require big changes (i.e., no JavaScript);
  • Can use arbitrary numbers of services.

Ideas for how to fix this:

  • Remove haproxy or increase its 8K GET header size limit (involves bigger platform infra changes);
  • Switch to POST for these requests and accept URLs won't be sharable (stops url being sharable);
  • Compress and base64 the data using JavaScript (requires JavaScript and changes to both the calculator and paas-billing);
  • Use POST but then have the data saved in a database and the user redirected to a sharable URL (disadvantage: another database to maintain.)
@46bit
Copy link
Contributor Author

46bit commented Apr 25, 2019

I'm not going to fix this right now because it is too big and there isn't a clear need. But it's worth remembering in case we work on the calculator again in future.

@richardTowers
Copy link
Contributor

richardTowers commented May 23, 2019

This was noticed by a GOV.UK PaaS tenant in this zendesk issue:

GOV.UK PaaS internal Zendesk link

@46bit
Copy link
Contributor Author

46bit commented May 23, 2019

We could quickly change the pricing calculator to only price up the first 12 items. Disable the links to add extra items and add a warning that for >12 items they should price it up again. I don't know if it's worth doing when the tenant figured that out for themself :)

@AP-Hunt
Copy link
Member

AP-Hunt commented Sep 4, 2019

Is this still a problem given @richardTowers' magic calculator?

@richardTowers
Copy link
Contributor

I seem to be able to get up to ~19 services now, then it starts going wrong (400 from HAProxy I think). Will be interesting to see whether this goes away when we play the story to remove HAProxy and use ALBs though.

@tlwr
Copy link
Contributor

tlwr commented Sep 4, 2019

I just tried hitting paas-admin using the ALB directly and I can get >21 services, didn't try more than that

@tlwr tlwr changed the title Calculator doesn't work for 13+ services Calculator doesn't work for 22+ services Dec 6, 2019
@tlwr
Copy link
Contributor

tlwr commented Dec 6, 2019

This is now a bug for 22 services since we deployed our request path changes with the ALB

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