Skip to content

Refactor ldap server #867

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
rimu-stack merged 22 commits into from
Jan 13, 2026
Merged

Refactor ldap server #867

rimu-stack merged 22 commits into from
Jan 13, 2026

Conversation

@Naksen
Copy link
Collaborator

@Naksen Naksen commented Dec 16, 2025 •
edited
Loading

Изменения

  1. Добавлен CONTEXT_TYPE во все LDAP запросы, чтобы избежать лишних вызовов resolve_deps.
  2. Перевод провайдеров в sync версии, чтобы разгрузить получение асинхронных зависимостей.
  3. [SearchRequest] Убрано безусловное получение зависимостей для аудита из контейнера.
  4. [SearchRequest] Убран лишний запрос на Directory.entity_type.
  5. [SearchRequest] Убран лишний joinedload(qa(Directory.user)).
  6. [SearchRequest] Подгрузка группы для атрибута member заменена c selectinload на джоин.
  7. [SearchRequest] stream_scalars -> scalars - изменение показало существенный прирост в скорости.

@Naksen Naksen added enhancement New feature or request python Pull requests that update Python code ldap labels Dec 16, 2025
Naksen added 21 commits January 13, 2026 14:59
@Naksen
refactor: change async functions to synchronous in MainProvider
86306f9
@Naksen
refactor: optimize query loading for entity type in SearchRequest
b62519d
@Naksen
refactor: change loading strategy for Directory group in SearchRequest
57354b2
@Naksen
refactor: change loading strategy from selectinload to joinedload for…
ef6e467 
… Directory group in SearchRequest
@Naksen
refactor: change async functions to synchronous in DHCP manager and A…
4c05c89 
…PI repository classes
@Naksen
refactor: change async iteration to synchronous for directory results…
62e2709 
… in SearchRequest
@Naksen
refactor: change loading strategy to use contains_eager for Directory…
416987a 
… entity in SearchRequest
@Naksen
refactor: reorganize import statements in search.py for clarity
e0465ed
@Naksen
refactor: add CONTEXT_TYPE class variable to LDAP request classes
ab973a8
@Naksen
refactor: LDAP context provider to use AsyncSessionSearchRequest and …
757a756 
…adjust scopes
@Naksen
test: enhance LDAP search request context provision with a dedicated …
4e0c379 
…method
@Naksen
refactor: optimize event processing logic in BaseRequest class
37fde5a
@Naksen
refactor: add async method to provide LDAP search request context
6941c94
@Naksen
refactor: add blank line for improved readability in LDAP context pro…
33a7d40 
…vider and request context files
@Naksen
refactor: specify type for ctx parameter in handle method of AbandonR…
2e4b692 
…equest class
@Naksen
refactor: remove debug logging for group membership in SearchRequest …
8d578da 
…class
@Naksen
refactor: update context handling in BaseRequest class for improved p…
165e9f0 
…rotocol operation management
@Naksen
refactor: change get_search_request_context method to synchronous
848ef18
@Naksen
refactor: change search context scope back to request
406426d
@Naksen
test: change search request context scope to request
ed35abe
@Naksen
add: abandon request context stub
dbcea97
@Naksen Naksen force-pushed the refactor_ldap_server branch from d473ce8 to dbcea97 Compare January 13, 2026 12:00
@rimu-stack rimu-stack merged commit e8cc0ad into dev Jan 13, 2026
6 checks passed
@rimu-stack rimu-stack deleted the refactor_ldap_server branch January 13, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rimu-stack rimu-stack rimu-stack approved these changes

@TheMihMih TheMihMih TheMihMih approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request ldap python Pull requests that update Python code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Naksen @rimu-stack @TheMihMih