Matrix table overflow due to huge version number

[ertrzyiks]

Pre issue-raising checklist

I have already (please mark the applicable with an x):

• Upgraded to the latest Pact Broker OR
• Checked the CHANGELOG to see if the issue I am about to raise has been fixed
• Created an executable example that demonstrates the issue using either a:
  • Dockerfile
  • Git repository with a Travis or Appveyor (or similar) build

Software versions

• pact-broker docker version: 2.79.1

Expected behaviour

Long version numbers are abbreviated

Actual behaviour

Only hash commits with 40 characters are abbreviated

Other version formats are rendered raw (We use a compound versioning scheme for the graphql gateway verification.)

Steps to reproduce

Publish a contract using a long version string

[TimothyJones]

Thanks for the report! I particularly enjoyed the original phrasing of "ridiculously long version number" :)

As a workaround (assuming the ridiculously long version number was in error), you could delete the version. I do agree it would be good to have this case handled in a more user friendly way, though.

[ertrzyiks]

I edited the description to make it look more professional as something that needs more attention than jokes. (However I still find hundreds character long version ridiculously long 😅 )

Please let me know if I can help to address this one, I came up with two ideas that I can contribute with:

• update AbbreviateNumber to have some hard limit anyway, not only 40 -> 6 hashes
• use fixed table-layout for matrix table so it never grows wider than 100% of the screen width

[bethesque]

I think we update AbbreviateNumber. If you'd like to contribute a fix, I'm happy to assist.

[ertrzyiks]

I prepared a straightforward fix in #511, please advise if we need anything else

[ertrzyiks]

Fixed in 2.88