implementation for an artifactory service.

[cdancy]

Looking for feedback on this impl and whether or not I'm going down the correct path and if this is something you all would be interested in taking in. I more/less just copied a lot of what I saw others doing in other services and did the same here to get things up and going quick. Tests still need to be written but the initial impl works as expected with no issues to speak of.

The service itself is basically just a wrapper around Artifactory's latest version endpoint the documentation of which you can FIND HERE.

@calebcartwright let me know what you think and if this is something you all would consider moving forward on. If not, and we had to maintain this out-of-tree, is there a good process for doing so other than the obvious you all would recommend?

[github-actions]

	Warnings
⚠️	This PR modified service code for artifactory but not its test code. That's okay so long as it's refactoring existing code.

	Messages
📖	✨ Thanks for your contribution to Shields, @cdancy!

Generated by 🚫 dangerJS against 00443c9

[cdancy]

No matter what I did I couldn't get any of this to show up in the local "integ test server". I assume I'm doing something wrong but filled out all the relevant links/options/docs/etc anyway if only for completeness sake.

[cdancy]

Don't know if you all care about keeping this in but it was generally useful for me when debugging to see what was coming through as values for the query parameters. Let me know if you want me to rip it out.

[cdancy]

@calebcartwright thoughts one way or another? I can continue with writing some integration tests (unless you have units which would be easier) but wanted to get your thoughts/opinions one way or another or just those from the community at large.

[calebcartwright]

One significant issue with this is that, afaik, there's no default public instance of Artifactory that we can rely on for our integration and testing. As such, this is a badge that seems like it would only be applicable for self-hosted instances and which would have no utility on the primary Shields.io instance.

This is something the Shields team has historically objected to doing, so we'd need some confirmation from other members of the team that they'd be okay with breaking precedent here before considering moving forward

[calebcartwright]

As far as the implementation goes, I feel like it's going in a vastly different direction than our typical services and I'm not entirely sure why that would need to be the case.

I'd suggest reviewing some of our material, linked below for reference, to reconsider the implementation or to at least help inform a subsequent explanation as to why this would require taking such a different path:

• https://github.com/badges/shields/blob/master/doc/TUTORIAL.md
• https://github.com/badges/shields/blob/master/doc/authentication.md
• https://github.com/badges/shields/blob/master/doc/adding-new-config-values.md

[chris48s]

Couple of things:

1. I don't think its worth getting into the implementation details at this stage. If a possible outcome here is "we don't want to add badges for this service", putting more work into the code isn't a useful thing to do yet. I don't want to waste anyone's time.

2. Maybe this is a good one to talk about next time we have a call. I think I'm still 👎 on accepting badges where there is no public instance and would only be relevant to self-hosting users. The thing that give me most pause about taking on the maintenance and support for something like this is that if there is no public instance of this API on the internet, it is basically impossible for me to test that this code "works".

One thing this does make me think is: This isn't super common, but also the idea of a badge that is only applicable to self-hosting users is something that has come up before. I wonder if there is a way we could provide a plugin hook to allow self-hosting users to conveniently inject a custom service like this without having to maintain a fork. 🤔 Hmmmm..

[cdancy]

I wonder if there is a way we could provide a plugin hook to allow self-hosting users to conveniently inject a custom service like this without having to maintain a fork.

@chris48s gets is ;)