alitheg commented on issue #944: URL: https://github.com/apache/tooling-trusted-releases/issues/944#issuecomment-4143295147
In fact, we already do have a `util.` function that tells us if LDAP is configured. Based on our conversation yesterday, it sounds like we should unify some checks into 3/4 distinct modes: - Production - Development - Development without LDAP - CI The last two could be the same, a "test" mode perhaps. I also suggested having a function which runs on server startup and if it detects production mode will run some explicit checks like "is ALLOW_TESTS turned off" and "is LDAP configured" and if not it would bail out of the server start. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
