Project

General

Profile

Feature #24127

Update AD and LDAP endpoints in API

Added by Erin Clark over 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Critical
Assignee:
Vladimir Vinogradenko
Category:
Middleware
Target version:
Estimated time:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

ldap_netbiosname_a is a required field for updating but is not returned by GET so the form is blank on that line,
will need ldap_certfile (or ldap_certificate?) as well, also ldap_netbiosaliases


Related issues

Related to FreeNAS - Feature #24103: Create LDAP UIResolved2017-05-23
Related to FreeNAS - Feature #24275: Active Directory UIResolved2017-05-31
Related to FreeNAS - Feature #26230: api param/calls for missing fields Closed: Behaves correctly2017-10-16

Associated revisions

Revision 6771a45a (diff)
Added by Vladimir Vinogradenko about 1 year ago

fix(gui): Use DojoModelFormResourceMixin to properly render extra form fields values for AD and LDAP API resources

Ticket: #24127

Revision 2c4d9f5e (diff)
Added by Vladimir Vinogradenko about 1 year ago

Dojo model form resource mixin + resolve ticket: #24127 (#362)

  • feat(gui): Add DojoModelFormResourceMixin that is able to render extra form fields values into API responses
  • fix(gui): Use DojoModelFormResourceMixin to properly render extra form fields values for AD and LDAP API resources

Ticket: #24127

  • Updated description and list of the fileds for AD and LDAP endpoints.
    Remove section of obsolete NT4 endpoint.

History

#1 Updated by William Grzybowski over 1 year ago

  • Assignee changed from William Grzybowski to John Hixson

#2 Updated by John Hixson over 1 year ago

  • Status changed from Unscreened to Screened

#3 Updated by Erin Clark over 1 year ago

  • Parent task deleted (#24103)

#4 Updated by Erin Clark over 1 year ago

#5 Updated by Erin Clark over 1 year ago

this also happens with AD, I am missing ad_netbiosname_a, ad_netbiosalias and ad_certificate. I am wondering if I shouldn't grab the netbios things from cifs since they have been unified under there but that could possibly be done more efficiently on the backend too

#6 Updated by Vaibhav Chauhan over 1 year ago

  • Target version changed from 11.0-U1 to 11.0-U2

#7 Updated by Vaibhav Chauhan over 1 year ago

  • Target version changed from 11.0-U2 to 11.0-U3

#8 Updated by John Hixson over 1 year ago

  • Status changed from Screened to Unscreened
  • Assignee changed from John Hixson to Timur Bakeyev

Timur, here is some low hanging fruit to help you start learning FreeNAS ;-)

#9 Updated by Timur Bakeyev over 1 year ago

  • Status changed from Unscreened to Screened

#10 Avatar?id=14398&size=24x24 Updated by Kris Moore over 1 year ago

  • Status changed from Screened to 46
  • Target version changed from 11.0-U3 to 11.1

Timur - Is this one you can still take a shot at?

#11 Updated by Timur Bakeyev over 1 year ago

  • Status changed from 46 to Investigation

I'm still looking into the ways to make it right, this involves some understanding of Django and MVC modelling there. Hope I'll do it, but if you need to rush it - some extra help could be nice.

#12 Updated by Timur Bakeyev over 1 year ago

  • Tracker changed from Bug to Feature

Actually this is a feature request, rather than a bug. With recent Django tickets I got a better understanding how to implement this one.

#13 Updated by John Hixson over 1 year ago

Timur Bakeyev wrote:

Actually this is a feature request, rather than a bug. With recent Django tickets I got a better understanding how to implement this one.

timur, this is pretty straight forward. The fields in the forms need to be moved to the model.

#14 Updated by Vaibhav Chauhan about 1 year ago

  • Priority changed from Expected to Critical

Hey Timur we need this for our new GUI to edit LDAP and AD fields. bumping up Priority for BETA.

#15 Updated by Vaibhav Chauhan about 1 year ago

ping on this ticket

#16 Updated by Vaibhav Chauhan about 1 year ago

#17 Updated by Timur Bakeyev about 1 year ago

#18 Updated by Timur Bakeyev about 1 year ago

VB, just for clarity, you need to get/set fields:

ldap_netbiosname_a
ldap_netbiosname_b
ldap_certificate
ldap_netbiosaliases

And their respective ad_ equivalents from the cifs_.

#19 Updated by Timur Bakeyev about 1 year ago

  • Status changed from Investigation to Reviewed by Developer

VB, thanks to Vladimir, the requested fields should be available now in the master:

{
  "ad_allow_dns_updates": false,
  "ad_allow_trusted_doms": false,
  "ad_bindname": "Administrator",
  "ad_bindpw": "passw0rd",
  "ad_certificate": null,
  "ad_dcname": "ad.freenas",
  "ad_disable_freenas_cache": false,
  "ad_dns_timeout": 60,
  "ad_domainname": "ad.freenas",
  "ad_enable": true,
  "ad_enable_monitor": false,
  "ad_gcname": null,
  "ad_groupdn": "",
  "ad_idmap_backend": "rid",
  "ad_kerberos_principal": null,
  "ad_kerberos_realm": 2,
  "ad_ldap_sasl_wrapping": "plain",
  "ad_monitor_frequency": 60,
  "ad_netbiosalias": "",
  "ad_netbiosname_a": "fn110-sdk",
  "ad_nss_info": null,
  "ad_recover_retry": 10,
  "ad_site": "",
  "ad_ssl": "off",
  "ad_timeout": 60,
  "ad_unix_extensions": false,
  "ad_use_default_domain": false,
  "ad_userdn": "",
  "ad_verbose_logging": true,
  "id": 1
}

#20 Updated by Dru Lavigne about 1 year ago

  • Subject changed from LDAP GET needs to return a couple fields to Update AD and LDAP endpoints in API
  • Status changed from Reviewed by Developer to Ready For Release
  • Assignee changed from Timur Bakeyev to Vladimir Vinogradenko

#21 Updated by Dru Lavigne about 1 year ago

  • Target version changed from 11.1 to 11.1-RC1

#22 Updated by Dru Lavigne about 1 year ago

  • Status changed from Ready For Release to Resolved

#23 Updated by Joe Maloney about 1 year ago

  • Needs QA changed from Yes to No
  • QA Status deleted (Not Tested)

Also available in: Atom PDF