Skip to content
This repository has been archived by the owner on Nov 3, 2017. It is now read-only.

Remove Duo source from CAS-MFA code #120

Open
jtgasper3 opened this issue Sep 24, 2015 · 1 comment
Open

Remove Duo source from CAS-MFA code #120

jtgasper3 opened this issue Sep 24, 2015 · 1 comment

Comments

@jtgasper3
Copy link

Duo code does not belong in CAS-MFAS. It puts an unrealistic burden upon the project to publish updates when Duo changes the internal workings of their code as they recently did. We don't do this for hibernate, spring etc, code. Why do it for Duo?

One of two things (and probably both) should happen:

  1. Provide an example of how to configure the maven overlay to compile the duo source and package the static assets as source-based customizations to CAS which is truly how Duo wants it treated.
  2. We publish the bits under a standalone library/war, but within our namespace. And put a big disclaimer that this is not an official library. I'd even add a line of static class initialization code that writes something to stdout that says as much. Then this package is referenced in the CAS-MFA' Duo pom.xml as "provided" so it is available for compile-time referencing, etc, but not formally included. Then folks can either do point 1 or add a reference to this item (point 2), but it is a formal decision on their part.
@jtgasper3
Copy link
Author

I know this may not be a popular opinion, but I'm requesting this in the mindset of a library owner/maintainer and not an consulting implementor. During client implementations, the client should be asked to contact Duo and read the provided prompt about why they have to compile Duo source instead of using compiled libraries. :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant