Skip to content

Conversation

@nazarewk
Copy link
Contributor

@nazarewk nazarewk commented Jul 30, 2025

image

@nazarewk nazarewk changed the title feat: add Routing Peer Access Policies diagram add Network Routing Peer diagram explaining Access Policies Jul 30, 2025
@nazarewk nazarewk force-pushed the feat/explain-routing-peer-policies branch from c192efa to 2542578 Compare July 30, 2025 15:33
@nazarewk
Copy link
Contributor Author

decided to include the original diagram sources so it's easier to adjust later on

@nazarewk nazarewk changed the title add Network Routing Peer diagram explaining Access Policies explain Access Policies to and through the Routing Peers Jul 30, 2025
@nazarewk nazarewk force-pushed the feat/explain-routing-peer-policies branch 4 times, most recently from 8b6c052 to ecba07f Compare August 1, 2025 15:44
@nazarewk nazarewk changed the title explain Access Policies to and through the Routing Peers new page: Networks access management Aug 4, 2025
@nazarewk nazarewk force-pushed the feat/explain-routing-peer-policies branch from c06528f to a2d8513 Compare August 4, 2025 15:56
@nazarewk nazarewk requested a review from braginini August 4, 2025 15:57
@nazarewk nazarewk force-pushed the feat/explain-routing-peer-policies branch from a2d8513 to 98d6696 Compare August 4, 2025 15:58
@nazarewk nazarewk force-pushed the feat/explain-routing-peer-policies branch 2 times, most recently from 92ebdf0 to 8a58d5c Compare August 5, 2025 07:59
@Blackclaws
Copy link

Related: netbirdio/netbird#4606

However I'd like to bring up the question whether the current behavior even makes sense if you need a whole page dedicated to explaining it.

It just took me an hour to debug that the input chain isn't populated because to me it was obvious that granting access to a subnet would also include the routing peer itself.

@nazarewk
Copy link
Contributor Author

nazarewk commented Nov 3, 2025

However I'd like to bring up the question whether the current behavior even makes sense if you need a whole page dedicated to explaining it.

@Blackclaws I understand the frustration (was there myself at least twice), but it's not as much about preserving the current behavior (it is what you get uniformly across all of the operating systems by default) as need to implement workarounds as fully fledged (and probably quite time-consuming) cross-platform feature that we don't have the manpower to properly design and implement in foreseeable future.

@nazarewk nazarewk closed this Nov 3, 2025
@nazarewk nazarewk deleted the feat/explain-routing-peer-policies branch November 3, 2025 18:38
@nazarewk nazarewk restored the feat/explain-routing-peer-policies branch November 3, 2025 18:38
@nazarewk nazarewk reopened this Nov 3, 2025
@Blackclaws
Copy link

However I'd like to bring up the question whether the current behavior even makes sense if you need a whole page dedicated to explaining it.

@Blackclaws I understand the frustration (was there myself at least twice), but it's not as much about preserving the current behavior (it is what you get uniformly across all of the operating systems by default) as need to implement workarounds as fully fledged (and probably quite time-consuming) cross-platform feature that we don't have the manpower to properly design and implement in foreseeable future.

Understood. Maybe adding a heads up to the networks page when you add resources that explains that routing peers by definition are not included in the scope whatever it may be, would alleviate that problem :)

I get that you need to be picky about what to implement and don't have unlimited resources. If this is something that gets implemented down the line that would be great however.

@nazarewk
Copy link
Contributor Author

nazarewk commented Nov 4, 2025

Understood. Maybe adding a heads up to the networks page when you add resources that explains that routing peers by definition are not included in the scope whatever it may be, would alleviate that problem :)

that's a good idea, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants