T7384: Radius patch for Dns-Server-IPv6-Address attribute - accel-ppp

[AlexIT-FT]

This adds the ability to pull IPv6 DNS servers from the radius response not just hard coded in the config file

Bit of background: https://www.hardill.me.uk/wordpress/2020/03/22/contributing-to-accel-ppp/

All credits to Ben Hardill

Change summary

Just to manage the attribute Dns-Server-IPv6-Address from Radius

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Code style update (formatting, renaming)
• Refactoring (no functional changes)
• Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
• Other (please describe):

Related Task(s)

T7384 accel-ppp: Radius patch for Dns-Server-IPv6-Address attribute

Related PR(s)

Checklist:

• I have read the CONTRIBUTING document
• I have linked this PR to one or more Phabricator Task(s)
• My commit headlines contain a valid Task id
• My change requires a change to the documentation
• I have updated the documentation accordingly

[github-actions]

❌
PR title does not match the required format

[sever-sever]

It is a nice practice to create a feature request from https://vyos.dev/
And refer to this task in the commit message and PR title.

[AlexIT-FT]

@sever-sever
sorry, I'm new to this kind of things, next time for sure
now, just waiting for acount approval in vyod.dev

[sever-sever]

Approved as it was tested by users, ref accel-ppp/accel-ppp#69

[dmbaturin]

@AlexIT-FT Your vyos.dev account is active. Could you please create a task and include its number in the commit message and the PR title? Then we can merge the PR.

[AlexIT-FT]

Task T7384 created at vyos.dev

[dmbaturin]

The title is good now. The commit message also needs to include the task number, and it's better to make it descriptive, like T7384: add a patch to support the DNS-Server-IPv6-Address attribute

[AlexIT-FT]

The title is good now. The commit message also needs to include the task number, and it's better to make it descriptive, like T7384: add a patch to support the DNS-Server-IPv6-Address attribute

I'm sorry for my ineptitude, but it's my first time.
If I click on the "commit" label (on top of this page), I read my commit message (dated Mar 13, 2025) but it seems to be readonly, I can't edit it.
What should I do to modify the commit message.

[sever-sever]

What should I do to modify the commit message.

In the git tool (console/terminal)

git commit --amend
git push --force

[AlexIT-FT]

The title is good now. The commit message also needs to include the task number, and it's better to make it descriptive, like T7384: add a patch to support the DNS-Server-IPv6-Address attribute

I tried many times in several ways (cli, web, ...) but the result is always the same.

The error is always relate to the very first commit message.

Commit message 'Create 0002-Radius-Dns-Server-IPv6-Address.patch' does not match the required format!

What am I messing up ?

Changes are already addressed.

[dmbaturin]

The patch is already tested in rolling. The PR is now compliant with contributing guidelines so I see no reasons not to merge it.