-
-
Notifications
You must be signed in to change notification settings - Fork 191
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
Add EfficientIP SOLIDServer plugin #582
base: main
Are you sure you want to change the base?
Conversation
Just a heads up. I reached out to EfficientIP to see if I could procure some sort of trial/demo. Still waiting on a response. |
Adding a quick link to #486 which this PR would address. |
Thank you for contacting them. I hope they see the value in it. |
I created a support ticket to request approval of your inquiry. Maybe if others interested do the same, that will help. Thanks again for all of your great work on Posh-ACME. |
I've still heard nothing but crickets from EfficientIP about a demo/trial. I'm tempted to reach out from my work address, but that probably wouldn't happen until the new year. Might you have access to installation media that could somehow find its way to me? I don't know how licensing works, so it may be pointless if it doesn't work without some sort of keys though. We run Infoblox at work which uses 60-day trial licenses by default. So I'm hoping there's something like that baked in. |
My case was closed on 12/31 noting that they will reach back out to me in Q1 with an update and also provided an email address for a Project Manager. At the moment, until I hear back from them, I will be unable to provide any bits they have behind their paywall - I'm sorry but will continue to follow-up. At the very least, I'll attempt to request the ability for me to setup a lab with temp license and give you access. No idea what the timeline will look like for that. If any others out there using EfficientIP SOLIDServer are reading this - feel free to open up a support ticket as it will likely get more traction with more requests. |
@rmbolger could be get in touch privately ? I can certainly share access to a temporary public SOLIDserver instance. Kind regards |
Thank you so much Alexis! The SOAP/REST API guide will be a good doc to share with Ryan as well. I really appreciate you helping out with this request! |
@alexissavin Sure. my email is ryan-oss {at} xyto.cc. |
Resolved issue with Entrust wildcard certificate acquisition noted here. rmbolger#567
Hey @jamiekowalczik. Just letting you know @alexissavin got me access to a test instance and I've started digging in a bit. One thing we discussed on our call was likely renaming the plugin to Other things I'm thinking about as I start poking around:
|
That all sounds great to me. I trust your direction and will adapt. Thank
you for supporting SOLIDServer EfficientIP!
If you need me to test anything, just let me know. Fwiw - i am using powershell core running on a Linux VM ( crazy, I know :) )
…On Tue, Jan 28, 2025, 7:54 PM Ryan Bolger ***@***.***> wrote:
Hey @jamiekowalczik <https://github.com/jamiekowalczik>. Just letting you
know @alexissavin <https://github.com/alexissavin> got me access to a
test instance and I've started digging in a bit. One thing we discussed on
our call was likely renaming the plugin to SOLIDServer instead of
EfficientIP to align better with their branding of the product.
"EfficientIP" will still be in the docs though.
Other things I'm thinking about as I start poking around:
-
Probably going to combine the username/password params into a single
PSCredential param.
-
Going to try and implement their Token authentication option if I can.
The tricky part is that it requires SHA3_256 hashing support which only has
native PowerShell support in in very recent PowerShell versions and relatively
recent versions of Windows 11 and Linux
<https://learn.microsoft.com/en-us/dotnet/standard/security/cross-platform-cryptography#sha-3>.
We'll see how that goes.
—
Reply to this email directly, view it on GitHub
<#582 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJNLHEQ5IFO5OWO7MJKGWD2NARFZAVCNFSM6AAAAABRN7OH3OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMRQGM3DAMJTGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Valuable proposal :)
I just made some comments in the code that I hope will be useful.
Additionally, one thing could be improved: validate that the DNS server specified can actually be used to push the change. Ideally write operation should only be done on the smart, or one standalone server. This can be achieved by considering the value of the vdns_parent_id property of all objects server/view/zone/rr.
Kind regards
Thanks for the suggestions, @alexissavin. In my local copy, I've already added a way to make cert validation and the View optional. I was considering trying to make the Server/dns_name parameter optional as well and find the zone via one or more Regarding the parent id property, would it be safe to only allow zones where The goal is that the View and Server only become mandatory if the user's environment has multiple copies of the same zone (like Internal and External) that would make picking one automatically ambiguous. |
Interesting observation as I was messing with stuff tonight. If you do a Seems like a potential bug. Feels like it should either throwing an error when the add parameters would be make the destination zone ambiguous or return all IDs of the records created. |
… and view are optional, improved error handling and logging
I managed to get token auth working despite the SHA3-256 library limitations by falling back to BouncyCastle for environments that don't support the .NET native version. You can basically just use your API Token+Secret as the username and password for the
I'm pretty happy with the overall state of the plugin at this point. Now I just need to write the user guide. |
Before I merge this, @jamiekowalczik can you test the updated version in your environment and let me know if everything still works for you? Also pinging @amckinney92 and @tall27 who were also interested in this over in #486. I'd love to get more testing in as many real environments as I can. |
Thanks @rmbolger. I confirmed that username/password as well as token auth both work well in my environment! |
This is my first pull request, I apologize if I am doing it incorrectly. As recently previously stated in your Issues by someone else, the quality and documentation you have put into this project is truly amazing.