Skip to content

Add missing value of NETWORK_PROTOCOL context variable #170

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.

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

Closed
wants to merge 1 commit into from

Conversation

zanseb
Copy link

@zanseb zanseb commented Jan 31, 2022

According to FirebirdSQL's core code 'TCPv6' is a viable value for the NETWORK_PROTOCOL context variable.

@zanseb
Copy link
Author

zanseb commented Jan 31, 2022

I have seen that WNET was removed 26days ago. I'm not sure whether it is safe to reflect that change in the current documentation.

According to FirebirdSQL's core code 'TCPv6' is a viable value for the NETWORK_PROTOCOL context variable
@zanseb zanseb force-pushed the fix_networkprotokoll_spec branch from d21c9c6 to a7c2ba7 Compare January 31, 2022 14:59
@mrotteveel
Copy link
Member

Sorry that there has been zero response to this PR so far, somehow I had missed the mail notifications. Given it is pretty out-of-date by now, I'll manually incorporate the changes.

Regarding WNET, it was only removed in Firebird 5.0, so it still needs to be documented in 3.0 and 4.0.

@mrotteveel mrotteveel self-assigned this Oct 6, 2023
@mrotteveel
Copy link
Member

This was already addressed for fblangref50.

mrotteveel added a commit that referenced this pull request Oct 6, 2023
Incorporate changes from missed PR by zanseb
@mrotteveel
Copy link
Member

Fixed in:

  • fblangref30: 1.17
  • fblangref40: 2.9

Given this is a minor fix, I'll deploy this later when more fixes have accumulated in these versions.

Thanks for reporting, and once again, apologies for the delay.

@mrotteveel mrotteveel closed this Oct 6, 2023
@zanseb
Copy link
Author

zanseb commented Oct 6, 2023

Thank you 🙂

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.

2 participants