Missing roles (SYSDG)

[PeterP55P]

Hi I can see that only SYSDBA and SYSOPER roles are supported at the moment. We need other roles to be supported as well. At the moment we are struggling with missing SYSDG role as we need to monitor Standby database that is in MOUNT state.
I can see it as simple adjustment and I would really appreciate if you can add it as soon as possible.
Thank you in advance.

Peter

[markxnelson]

Hi @PeterP55P thanks for your suggestion, we will take a look and get back to you soon.

[markxnelson]

todo - add other suth modes as per https://odpi-c.readthedocs.io/en/latest/enums/dpiAuthMode.html#dpiauthmode

[markxnelson]

looks like this may potentially require a change in godror too - see https://github.com/godror/godror/blob/main/dsn/dsn.go#L163

[PeterP55P]

Hi @markxnelson thanks for quick answer. I was a bit worried that this change will require changes also in other projects. I have opened a feature request godror/godror#360 to implement SYSDG. I hope someone pick it up so fast as you did.
Thanks,
Peter

[markxnelson]

Hi @PeterP55P - godror have implemented the needed code - see this commit:

godror/godror@67aeda6

We will pick this up - if you are ok to build from main and test it, that'd be great. We are working on one other issue that I want to fix before we do a new release #177 - hoping to get that fixed in the next day or three as well.

[PeterP55P]

Yep I saw that. I'm glad that they also act so quickly.

I'll be at my computer earliest on Monday so if you'll release new version by then that would be great. If not I'll try to compile it on my own.
Thank you.

[markxnelson]

Ok, they have not done a release yet. I see in their changlog, they added this to 0.47.0. I can point to the commit, which would be ok for testing, but for a release, I'd prefer to point at one of their releases, not just a 'random' commit. I'll reach out to them to ask when they plan to release.

[PeterP55P]

That sounds good. Thanks

[markxnelson]

FYI, pushed commits in branch 180 which should provide what you need.

[PeterP55P]

Thanks

[markxnelson]

fyi changes are in main now

[markxnelson]

Fixed in 1.5.4 - please let us know when you have a chance to test if it meets your needs. Thanks very much for your input.

[PeterP55P]

We did a test and it works perfectly. Thank you @markxnelson

[markxnelson]

Great, thanks for letting us know. Please don't hesitate to open an issue if you have any other problems or suggestions!