Hi Will, Can you try this patch please? https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff <https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff>
Apply it by downloading it and then using patch, i.e: # cd /usr/local/pf # wget https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff <https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff> # patch -p1 < b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff <https://github.com/inverse-inc/packetfence/commit/b9642f12ed9bd3ec62f800bd4a5dfd36702553c2.diff> # bin/pfcmd service pf restart Then you can try deleting the source from the GUI and then recreating it again. If it works we've got ourselves a fix. Regards, -- Louis Munro lmu...@inverse.ca <mailto:lmu...@inverse.ca> :: www.inverse.ca <http://www.inverse.ca/> +1.514.447.4918 x125 :: +1 (866) 353-6153 x125 Inverse inc. :: Leaders behind SOGo (www.sogo.nu <http://www.sogo.nu/>) and PacketFence (www.packetfence.org <http://www.packetfence.org/>) > On Aug 14, 2017, at 14:29, Rossing, Will <wross...@css.edu> wrote: > > Hey Louis, Yes I was just going to report that that works, just comma > separation, the GUI won't show the list but it still works. Thanks for > your reply! > > On Mon, Aug 14, 2017 at 12:37 PM, Louis Munro <lmu...@inverse.ca > <mailto:lmu...@inverse.ca>> wrote: > Hi Will, > This looks like a bug from the GUI that saves the list of carriers the wrong > way. > > Can you try to change the source to this (manually edit the file): > > > [sms] > description=SMS-based registration > sms_carriers=100061,100107 > type=SMS > create_local_account=no > set_access_level_action= > local_account_logins=0 > > Then run this command: > > # /usr/local/pf/bin/pfcmd configreload hard > > > > And try again? > > The bug, if that's what it is, is in the code that saves the config. > So editing the file and reloading it should be a (temporary) workaround. > > Please confirm if this works for you. > If it does we'll open an issue on GitHub for tracking and issue a maintenance > patch. > > > Regards, > -- > Louis Munro > lmu...@inverse.ca <mailto:lmu...@inverse.ca> :: www.inverse.ca > <http://www.inverse.ca/> > +1.514.447.4918 x125 <tel:(514)%20447-4918> :: +1 (866) 353-6153 x125 > <tel:(866)%20353-6153> > Inverse inc. :: Leaders behind SOGo (www.sogo.nu <http://www.sogo.nu/>) and > PacketFence (www.packetfence.org <http://www.packetfence.org/>) > >> On Aug 14, 2017, at 12:51, Rossing, Will via PacketFence-users >> <packetfence-users@lists.sourceforge.net >> <mailto:packetfence-users@lists.sourceforge.net>> wrote: >> >> More info, it works when we only put one SMS provider in the list, if we add >> more than one, it gets the exception error. >> This is how it writes multiple carriers to the config file and seems like it >> can't parse it properly or something: >> >> [sms] >> description=SMS-based registration >> sms_carriers= <<EOT >> 100061 >> 100107 >> EOT >> type=SMS >> create_local_account=no >> set_access_level_action= >> local_account_logins=0 >> >> >> >> One provider works: >> [sms] >> description=SMS-based registration >> sms_carriers=100107 >> type=SMS >> create_local_account=no >> set_access_level_action= >> local_account_logins=0 >> >> >> On Mon, Aug 14, 2017 at 10:59 AM, Rossing, Will <wross...@css.edu >> <mailto:wross...@css.edu>> wrote: >> Just deploying 7.2 to production and am getting the following when choosing >> the sms authentication in the captive portal. >> >> Caught exception in captiveportal::Controller::Root>dynamic_application >> "Can't call method "fetchall_arrayref" on an undefined value at >> /usr/local/pf/lib/pf/sms_carrier.pm <http://sms_carrier.pm/> line 88." >> >> I swear this worked last week when I put the box in production temporarily. >> I've tried removing and adding back in carriers, etc. Any ideas? I hate >> to have to roll back if I can avoid it! >> >> Thanks >> >> Will >> >> -- >> >> >> Will Rossing >> Manager, Network Services | 218.723.6729 <tel:(218)%20723-6729> | >> wross...@css.edu <mailto:wross...@css.edu> >> >> >> -- >> >> >> Will Rossing >> Manager, Network Services | 218.723.6729 <tel:(218)%20723-6729> | >> wross...@css.edu >> <mailto:wross...@css.edu>------------------------------------------------------------------------------ >> Check out the vibrant tech community on one of the world's most >> engaging tech sites, Slashdot.org <http://slashdot.org/>! >> http://sdm.link/slashdot_______________________________________________ >> <http://sdm.link/slashdot_______________________________________________> >> PacketFence-users mailing list >> PacketFence-users@lists.sourceforge.net >> <mailto:PacketFence-users@lists.sourceforge.net> >> https://lists.sourceforge.net/lists/listinfo/packetfence-users >> <https://lists.sourceforge.net/lists/listinfo/packetfence-users> > > > > > -- > > > Will Rossing > Manager, Network Services | 218.723.6729 | wross...@css.edu > <mailto:wross...@css.edu>
signature.asc
Description: Message signed with OpenPGP
------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________ PacketFence-users mailing list PacketFence-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/packetfence-users