unvendor keycloak_metrics_spi

[Adam-D-Lewis]

Reference Issues or PRs

Closes #1805

What does this implement/fix?

Put a x in the boxes that apply

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds a feature)
• Breaking change (fix or feature that would cause existing features not to work as expected)
• Documentation Update
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no API changes)
• Build related changes
• Other (please describe): See linked issue

Testing

• Did you test the pull request locally?
• Did you add new tests?

Any other comments?

[iameskild]

LGTM

[pmeier]

New implementation is a lot simpler, thanks Adam! Good to merge if CI is green.

[pmeier]

This seems like an artifact from the previous implementation that had this and the next line on one line.

Suggested change

	chown 1000:1000 /data/keycloak-metrics-spi-2.5.3.jar &&
	chown 1000:1000 /data/keycloak-metrics-spi-2.5.3.jar

[pmeier]

Nit: formatting looks a little off

Suggested change

	if ! (echo "$SHA256SUM /data/keycloak-metrics-spi-2.5.3.jar" | sha256sum -c)
	then
	echo "Error: Checksum not verified" && exit 1
	else
	chown 1000:1000 /data/keycloak-metrics-spi-2.5.3.jar &&
	chmod 777 /data/keycloak-metrics-spi-2.5.3.jar
	fi
	if ! (echo "$SHA256SUM /data/keycloak-metrics-spi-2.5.3.jar" | sha256sum -c); then
	echo "Error: Checksum not verified" && exit 1
	else
	chown 1000:1000 /data/keycloak-metrics-spi-2.5.3.jar &&
	chmod 777 /data/keycloak-metrics-spi-2.5.3.jar
	fi