Vorkout/connect
4
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

feat: settings, delete port validation #22

Open
ivan.dev wants to merge 3 commits from VORKOUT-31 into master
pull from: VORKOUT-31
merge into: Vorkout:master
Conversation 1 Commits 3 Files Changed 32 -31
1 changed files with 0 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c86b7eb624 - Show all commits

31
api/api/endpoints/ps_node.py
View File

@@ -61,21 +61,6 @@ async def delete_ps_node_endpoint(
details={"next_node_id": ps_node_delete_data.next_node_id}, details={"next_node_id": ps_node_delete_data.next_node_id},
) )
# is_connected = await check_node_connection(
# connection, ps_node_delete_data.node_id, ps_node_delete_data.next_node_id, int(ps_node_delete_data.port)
# )
# if not is_connected:
# raise create_validation_error(
# message="Node connection validation failed",
# status_code=status.HTTP_400_BAD_REQUEST,
# details={
# "node_id": ps_node_delete_data.node_id,
# "next_node_id": ps_node_delete_data.next_node_id,
# "port": ps_node_delete_data.port,
# },
# )
service = PsNodeService(connection) service = PsNodeService(connection)
mikhail.dev marked this conversation as resolved Outdated
mikhail.dev commented 2025-11-17 13:41:48 +05:00
Outdated
Review
Copy Link

Если это ненужный код, то может лучше удалить?

Если это ненужный код, то может лучше удалить?
ivan.dev commented 2025-11-18 12:15:16 +05:00
Outdated
Review
Copy Link

Кирилл сказал на прошлом созвоне, пока проверку по портам закомментировать

Кирилл сказал на прошлом созвоне, пока проверку по портам закомментировать
cyrussmeat commented 2025-11-24 19:54:37 +05:00
Outdated
Review
Copy Link

С т.з. репозитория можно удалить в этой версии, мы всегда можем вернуться к этой ветке и позаимствовать код, согласен с Михаилом - не надо его тащить про запас

С т.з. репозитория можно удалить в этой версии, мы всегда можем вернуться к этой ветке и позаимствовать код, согласен с Михаилом - не надо его тащить про запас
👍 2
try: try:
result = await service.delete(ps_node_delete_data.next_node_id) result = await service.delete(ps_node_delete_data.next_node_id)
@@ -153,22 +138,6 @@ async def create_ps_node_endpoint(
details={"parent_id": parent_id, "expected_ps_id": target_ps_id, "actual_ps_id": parent_node.ps_id}, details={"parent_id": parent_id, "expected_ps_id": target_ps_id, "actual_ps_id": parent_node.ps_id},
) )
# parent_port_number = node_instance_validated.parent_port_number
# parent_settings = parent_node.settings or {}
# available_port_numbers = []
# for key, value in parent_settings.items():
# if "port" in key.lower() and isinstance(value, int):
# available_port_numbers.append(value)
# if parent_port_number not in available_port_numbers:
# raise create_validation_error(
# message="Parent port number is invalid",
# status_code=status.HTTP_400_BAD_REQUEST,
# details={"parent_id": parent_id, "parent_settings": parent_settings},
# )
service = PsNodeService(connection) service = PsNodeService(connection)
try: try:
ps_node_front_response = await service.create(ps_node.data, ps_node.links, user_validation.id) ps_node_front_response = await service.create(ps_node.data, ps_node.links, user_validation.id)