Modify

Opened 7 years ago

Closed 6 years ago

#7744 closed enhancement (fixed)

base-files/ network config lib to include vendorid in udhcpc

Reported by: KadettGTE Owned by: developers
Priority: normal Milestone: Backfire 10.03.1
Component: base system Version: Backfire 10.03.1 RC1
Keywords: udhcpc vendorid Cc:

Description

Since some ISP's may require a vendor id in the dhcp request i made this small modification to the code in order to have a option 'vendorid' 'somevendor' under the interface config

Index: package/base-files/files/lib/network/config.sh
===================================================================
--- package/base-files/files/lib/network/config.sh (revision 22588)
+++ package/base-files/files/lib/network/config.sh (working copy)
@@ -354,6 +354,7 @@

config_get hostname "$config" hostname
config_get proto1 "$config" proto
config_get clientid "$config" clientid

+ config_get vendorid "$config" vendorid

config_get_bool broadcast "$config" broadcast 0


[ -z "$ipaddr" ]

@@ -368,6 +369,7 @@

${ipaddr:+-r $ipaddr} \
${hostname:+-H $hostname} \
${clientid:+-c $clientid} \

+ ${vendorid:+-V $vendorid} \

-b -p "$pidfile" $broadcast \
${dhcpopts:- -O rootpath -R &}

Attachments (1)

base-files.patch (701 bytes) - added by KadettGTE 7 years ago.
.patch file

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by KadettGTE

.patch file

comment:1 Changed 7 years ago by jow

Thew patch looks good. Those ISPs requiring a vendor id... is it enough to provide any id or has it to be a specific value? Maybe we can make the option to "OpenWrt".

comment:2 Changed 7 years ago by jow

I meant "default to OpenWrt"

comment:3 Changed 7 years ago by anonymous

my ISP for example requires 2WHPL, else it won't give you an address, but if you don't pass any option , it will just ignore it :)

comment:4 Changed 7 years ago by KadettGTE

erm, the previous post was me lol

comment:5 Changed 7 years ago by jow

  • Resolution set to fixed
  • Status changed from new to closed

Committed in r23279 - thanks!

comment:6 Changed 6 years ago by Morzzz

  • Resolution fixed deleted
  • Status changed from closed to reopened

'vendorclass' is being used in /etc/config/network, so 'vendorid' needs to be changed to 'vendorclass' in /lib/network/config.sh

comment:7 Changed 6 years ago by jow

  • Resolution set to fixed
  • Status changed from reopened to closed

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.