[Bug 63505] enhancement - support of stored procedures for DataSourceRealm authentication

2020-07-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63505

--- Comment #5 from Eugène Adell  ---
Exactly, both files are the same except I needed to override these 2 functions
and also add variables for handling the procedures names (and what comes with
them : getters/setters of course). Maybe making it a child class of
DataSourceRealm, but would it be really clean this way ?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Jakarta EE APIs

2020-07-25 Thread Raymond Auge
+1

- Ray

On Wed., Jul. 22, 2020, 5:29 p.m. Martin Grigorov, 
wrote:

>
>
> On Wed, Jul 22, 2020, 18:10 Mark Thomas  wrote:
>
>> Hi all,
>>
>> We currently have implementations for all of the Jakarta APIs that ship
>> with Tomcat and partial implementations for 5 additional Jakarta APIs that
>> are compile time only dependencies.
>>
>> I was checking those partial implementations earlier today when I noticed
>> the Jakarta Mail API needed updating to use generics. I started on that but
>> paused when it looked like a number of new (dummy) classes would be
>> required.
>>
>> Considering alternative options, I wondered about depending on the
>> Jakarta API JARs directly. This would be a return to the 5.5.x era approach
>> without  hopefully, the issue that JARs could be difficult to obtain.
>>
>> I have this implemented locally. It removes about 1000 lines of .java
>> files (although just under 10% of them are actual code) and adds about 100
>> lines of build file config and anither 50 of IDE configuration.
>>
>> With the Jakarta JARs being readily available in Maven Central I think
>> the primary issue that led to the current approach is no longer a concern.
>>
>> Thoughts on switching to using the JARs directly? I can provide a
>
>
> +1
>
> PR if that is helpful.
>
>
>> Mark
>>
>> -
>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>> For additional commands, e-mail: dev-h...@tomcat.apache.org
>>
>>