[pve-devel] intro'ing myself
Hi all, First of all nice to meet you, As both a new participant on the list aswell as someone who is trying to step in/up/learn-by-doing i expect my future posts to contain a lot of questions and struggles trying to find a/the structure. My (personal) goal is to enhance skills and produce a working oauth extension, which i have seen was already requested back in 2017. As PVE as product means multiple disciplines i hope i can be directed in the right direction, as currently i see the code (individually (per-file)), but i am lacking the linkage. I mean an extension of a base something i can understand/comprehend, but some stuff like schema seems to be burried deep somewhere and havent figured out how to add stuff. If i somehow violate/inappropriately ask or post, please do tell/explain to me :) best regards, Michael Honkoop ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
[pve-devel] Feature: OAuth / OpenID-connect implementation
Hi all, I am slowly working on implementing the authentication method/ entry fields for it in PVE, which have shown up, and look to be fine (at first glance). There are 2 things i am yes unable to grasp, and i hope you could give me some directions : 1) UI : As Oauth/OpenID-connect does not have a need for a field 'tfa', but its inherited from the common auth, i'd like to override it ( as in not visible) 2) Schema: I cant seem to get where a/the schema is defined, or updated for that matter. I mean i have the UI / auth on the dc level now showing an entry and when i select add it will spawn the dialog to input parameters. However if i were to put some text in there and click add it spawns an errormessage 'Parameter verification failed. (400)', 'type: value 'oauth' does not have a value in the enumeration 'ad, ldap, pam, pve'' and 'property is not defined in schema and the schema does not allow additional properties' i've been searching where i can define it but this still have not found, or better said due to my newness in this i'm not following where it is/should be defined. best regards, Michael Honkoop ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
Re: [pve-devel] Feature: OAuth / OpenID-connect implementation
Hi Fabian, Thanks for your hints regarding where to find the realm types, researching further on my issue. best regards, Michael Honkoop >>> Fabian Grünbichler 8:12 14-12-2020 >>> On December 13, 2020 2:16 am, Michael Honkoop wrote: > Hi all, > > I am slowly working on implementing the authentication method/ entry fields for it in PVE, which have shown up, and look to be fine (at first glance). > > There are 2 things i am yes unable to grasp, and i hope you could give me some directions : > > 1) > UI : > As Oauth/OpenID-connect does not have a need for a field 'tfa', but its inherited from the common auth, i'd like to override it ( as in not visible) > > 2) > Schema: > I cant seem to get where a/the schema is defined, or updated for that matter. > I mean i have the UI / auth on the dc level now showing an entry and when i select add it will spawn the dialog to input parameters. > However if i were to put some text in there and click add it spawns an errormessage 'Parameter verification failed. (400)', 'type: value 'oauth' does not have a value in the enumeration 'ad, ldap, pam, pve'' and 'property is not defined in schema and the schema does not allow additional properties' > > i've been searching where i can define it but this still have not found, or better said due to my newness in this i'm not following where it is/should be defined. the possible realm types are defined here: https://git.proxmox.com/?p=pve-access-control.git;a=blob;f=PVE/API2/Domains.pm;h=9c2b2548028790f19baaa81bde4cf0f071bb7719;hb=HEAD#l86 https://git.proxmox.com/?p=pve-access-control.git;a=blob;f=PVE/Auth/Plugin.pm;h=141305336c8640dd40879eb6e084b95c51ad0fa7;hb=HEAD https://git.proxmox.com/?p=pve-access-control.git;a=tree;f=PVE/Auth;h=935646bae7647759c8b9ac04881275affb184500;hb=HEAD each type registers as a plugin in the SectionConfig (which is in pve-common), the add/update/delete schemas are generated from the plugin definitions. > > best regards, > > Michael Honkoop > > > ___ > pve-devel mailing list > pve-devel@lists.proxmox.com > https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel > > > ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
Re: [pve-devel] Feature: OAuth / OpenID-connect implementation - II
Hi all, In watching build-behaviour, i notice that when i build another git repo (pve-manager) it drops the directory 'dest' and then ( if one deletes the resulting *buildinfo, *.changes and *.deb) starts rebuilding. On the pve-access-control it does not remove the directory 'libpve-access-control-6.1' , so when one has this dir, ( and assuming same behaviour as seen on pve-manager repo) it will take all sources from this dir - regardless of changes. Is there a specific reason this differs from the pve-manager repo ? - as only after analysing why my added file was skipped over and over again ( in pve-access-control ) has lead me to a lot of questionmarks in my mind. best regards, Michael Honkoop ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel