https://github.com/lldap/lldap logo
Join Discord
Powered by
# general
  • e
    phew
  • n
    And familiarity (or lack of)
  • e
    i feared worse
  • n
    You can already create or delete users
  • e
    ok, that's not was i was looking for lol
  • e
    ig i need to dust off my rust book lol
  • e
    i'll try to see what i can do, but i doubt i'll achieve much
  • n
    LLMs are getting pretty good
  • n
    And the project structure is fairly straightforward
  • n
    There's a LDAP -> internal operations translation layer
  • e
    if you mean to explain how things work, yea sure i do agree, but i refuse to make ai slop pull requests
  • n
    So it's just a matter of calling "update user" with the right arguments
  • n
    That's great since I wouldn't review them!
  • n
    But you can use ai without just writing slop
  • e
    I've only had one ai made pull request on my projects so far and it was honestly horrible lol
  • n
    Oh yeah, don't just delegate to an LLM, but let it create a first draft and iterate
  • e
    is this file out of date? cause either that or i'm blind, i don't see an infra and a domain folder... https://github.com/lldap/lldap/blob/main/docs/architecture.md#code-organization
  • n
    Aha yes
  • n
    It's been on my to-do to update it
  • n
    It's all in crates now
  • n
    Except the entry point, the server
  • n
    I think you can restrict your PR to the crates/ldap folder
  • e
    ah ok, makes sense
  • n
    We have a "do_modify_request" in handler.rs that you can follow/complete
  • n
    And I guess if you add support for custom attributes, you'll want to edit create_user as well
  • e
    i'll see
  • n
    And for reference you can check how GraphQL does it
  • e
    that's great actually
  • e
    thx
  • z
    🤔