Skip to content
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

Required field checks are missing at Setup TUI #754

Closed
quiver opened this issue Sep 5, 2021 · 1 comment
Closed

Required field checks are missing at Setup TUI #754

quiver opened this issue Sep 5, 2021 · 1 comment

Comments

@quiver
Copy link

quiver commented Sep 5, 2021

Environment

  • Gluu Server 4.2
  • OS : Ubuntu 20.04

Problem

At Setup TUI's "Gathering Information" step([1]), you're required to fill in fields like

  • Hostname
  • Support Email

Some fields are required(Support Email, Country Code, etc.), but some fields are optional(e.g. City or Locality).

If you make "City or Locality" field blank and click "Next" at the confirmation step, you can't go on to the next step.

Cntrl-C shows that the script is chalked at check_properties method.
See the traceback at the end of this issue.

When I fill in all fields at "Gathering Information", I could successfully finish the setup process.

How to fix

If "City or Locality" is mandatory, you should validate its existence.

Cntrl-C Traceback

Traceback (most recent call last):
  File "./setup.py", line 6083, in <module>
    GSA.run()
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/apNPSApplication.py", line 30, in run
    return npyssafewrapper.wrapper(self.__remove_argument_call_main)
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/npyssafewrapper.py", line 41, in wrapper
    wrapper_no_fork(call_function)
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/npyssafewrapper.py", line 97, in wrapper_no_fork
    return_code = call_function(_SCREEN)
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/apNPSApplication.py", line 25, in __remove_argument_call_main
    return self.main()
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/apNPSApplicationManaged.py", line 172, in main
    self._THISFORM.edit()
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/fm_form_edit_loop.py", line 47, in edit
    self.edit_loop()
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/fm_form_edit_loop.py", line 38, in edit_loop
    self._widgets__[self.editw].edit()
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/wgwidget.py", line 447, in edit
    self._edit_loop()
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/wgwidget.py", line 463, in _edit_loop
    self.get_and_use_key_press()
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/wgwidget.py", line 599, in get_and_use_key_press
    self.handle_input(ch)
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/wgwidget.py", line 71, in handle_input
    self.handlers[_input](_input)
  File "/usr/local/lib/python3.8/dist-packages/npyscreen/wgbutton.py", line 97, in h_toggle
    self.when_pressed_function()
  File "/install/community-edition-setup/pylib/tui.py", line 725, in nextButtonPressed
    self.parentApp.installObject.check_properties()
  File "./setup.py", line 1007, in check_properties
    self.city = input('City: ').strip()
KeyboardInterrupt

[1] https://gluu.org/docs/gluu-server/4.2/installation-guide/setup_py/#setup-prompt

devrimyatar added a commit that referenced this issue Sep 8, 2021
@devrimyatar
Copy link
Collaborator

Fixed

yurem added a commit that referenced this issue Oct 5, 2021
* fix post setup for scim

* Casa's script for DUO

* oxBiometricDevices, oxDUODevices

* Putting oxDuoDevices and oxBiometricDevices at the bottom

* oxBiometricDevices and oxDuoDevices are a part of gluuCustomPerson and not GluuPerson

* fix custom_schema.json

* change desc in custom_schema.json

* (4.2.2) setup: revoke interception script sample

GluuFederation/oxAuth#1502

* ask if oxtrust to be installed

* couchase user pfrefix in datasource

* fix version in post-setup

* Version 4.2.2.Final

* Version 4.2.3-SNAPSHOT

* (4.2.2) setup: added ou=stat,o=gluu

GluuFederation/oxAuth#1512

* (4.2.3) setup: added jansStatEntry OC

GluuFederation/oxAuth#1512

* (4.2.3) setup: added jansId to indexes

GluuFederation/oxAuth#1512

* change mod_ssl name for rhel7

* data type imapdata -> json for attrib 42E1

* ldap2cb: migrate to py3 and fixes

* Make some SCIM attributes multivalued as in spec

* Add multivalued data

* Version 4.2.3.Final

* (4.2.3) setup: added statWebServiceIntervalLimitInSeconds:60

GluuFederation/oxAuth#1512

* mod_ssl centos7

* Version 4.3.0.Final

* Add u2f and fido2 test data

* Add sample update_token script

* Add sample update_token script

* fixes templates

* show version on tui

* fixes

* no-chroot install script

* install missing packages

* Add keepAliveInterval CB SDK support

* F2 to display version info

* python3-six dependency for gluu_setup.py

* refactor: 4.3.0 setup

* fix: oxd-server.default

* fix: idp download

* fix: don't backup same file

* refactor: seperate argparser

* refactor(rdbm): json files

* fix: add ou=stat,o=gluu

* feat: migrate Jans to Gluu OC

* feat: migrate Jans to Gluu OC

* refactor: rdbm works

* feat(rdbm): installation

* fix: gluu_installer.py for args

* fix: radius installer

* fix(rdbm): local install

* fix(schema): add missing attributes

* fix(rdbm): schema and sql data types

* fix: gluuPerson

* fix: update package list

* fix: update installer for missing packages

* fix: ruamel module

* fix: add pylib path to encode.py

* refactor: re-order installers

* fix: encode.py

* fix: doc_id gluu --> _

* fix: remov unused file db_utils_org.py

* feat: add -n option to gluu_install.py

* fix: change ce setup branch

* feat: implement --no-progress

* fix: redhat8 installs

* feat: add -no-setup to installer

* fix: typo

* fix: enable opendj

* re-add stat service attrbiutes

* fix: generate schema

* fix: set jetty timeout 300

* fix: set systemd tiemout

* fix: spanner fixes

* feat: logging config

* fix: cb installation

* refactor: log filename db-backend.log

* feat: add --dist-server-base

* fix: couchbase test data loading

* fix: cn for uniqueness in ldap

* fix: load test data for ldap

* fix: test data loader ldap bind

* fix: dsconfig after test data

* fix: remove attrbiutes lifetime & salt

* fix: implement spanner test data loader

* fix: add del and exp to gluuPasswordResetRequest

* fix: re-generate schema

* feat: pre add base metric entries

* feat: pre add base metric entries

* feat: pre add base metric entries

* fix: spacing

* Update install.py

added sqlalchemy extraction to ces_dir

* fix: added ssnId to oxAuthUmaRPT

* fix: added ssnId to oxAuthUmaPCT

* fix: load ldif

* fix(wrends): change display text to opendj

* fix: spanner subtables

* fix: couchbase install (ref: #741)

* faet: review Spanner indexes #736

* fix: remote couchbase arg

* faet: review Spanner indexes #736

* fix: mysql indexes (ref: #737)

* fix: chown root:gluu gluuOptPythonFolder

* fix(spanner): passport installation (ref: #740)

* fix: creating o=metric related entries (ref: #742)

* fix: ldap test data loader

* fix: spanner test data loading

* feat(test-data): display if test data will be loaded

* feat(cert): download Apple WebAuthn Root CA

* fix: setting couchbase admin password

* fix: passport cert files mode

* feat: merge test config changes from jans

* fix: saml couchbase install (ref: #741)

* fix: check if apache module is availabe before enabling (ref: #741)

* feat: check user and group before adding

* fix: remove wrong error line for rendering ecnode script

* feat: check if opendj ports are free (ref: #743)

* fix: /etc/certs permissions

* feat: fix attribute names to conform Gluu shema

* feat: update test CIBA configuration

* feat: update test CIBA configuration

* fix: test data additional columns creation

* feat: fix configuration entry DN in server tests

* fix: limit lenght of description for indexing

* fix: set size 768 for description

* fix: spanner test data columns

* fix: cb test oxauth config

* feat: update default server profile

* feat: update default server profile

* feat: update default server profile

* fix: don't create gluuCustomPerson

* fix: typo

* feat: update default server profile

* feat: command line backend options

* fix: set couchbase host to hostname for local installation

* fix: disable ssl ofr cb test profile

* feat: merge scim properties from Jans

* fix: couchbase hostname

* fix: rdbm test data columns

* fix: local mysql installation

* fix: typo

* fix: prepend plus sign GluuFederation/casa#138

* feat(tui): implement backends

* fix: oxd server gluu storage config

* fix: typo

* fix: oxd-server progress string

* fix: remote cb install

* fix: remove tmp file

* fix: create UMA SCIM resource (ref: #744)

* fix: typo

* fix: double backup when inserting lines to file

* fix: scim-rp.jks goes to bot output and certs dir

* feat: file descriptor limits for systemd services (ref: #734 #745 )

* fix: post install tasks

* refactor(backend): disable rdbm

* feat: re-try download three times on fail

* Load module before config 

https://support.gluu.org/other/9790/bug-in-httpdconf/

* fix: passport certs ownerships

* fix: remove sqlalchemy related code from install.py

* fix: remove gluu_install.py if extracted within container

* fix: chenage jetty version

* fix: typo

* Updated marketing messages

Updated marketing messages

* Revert "refactor(backend): disable rdbm"

This reverts commit e204f11.

* fix: re-do commit 0fc53c8

* fix: re-do commit 1ff6a74

* refactor(rdbm): suppress rdbm options

* fix: scim istallation

* fix: ownership of webapps dir (ref: #746)

* feat(setup4.3): added stat scope

GluuFederation/oxAuth#1554

* refactor(4.3setup): renamed scope stat -> jans_stat

GluuFederation/oxAuth#1554

* fix: centos packages

* chore: sync scim script wrt oxexternal

* chore: remove comment

* feat: add enable war updates to gluu_install.py

* feat: extract sqlalchemy

* feat: gluu_install.py update oxd-server

* feat: dummy installation

* feat: added o to oxAuthClient

* fix: typo on help

* fix: update shibboleth idp custom script

* chore: adjust length casa attributes

* fix: don't create config for oxd

* feat: Touch ID as a fido2 device (platform authenticator)

* feat: Adjust protection mode handling in SCIM

See GluuFederation/scim#20

* feat: scim scopes (ref: #750)

* feat: UMA mode for SCIM (ref: #752)

* feat: dbUtils set config by dn

* fix: find oxd_host when collecting properties

* feat: fido installer add do_import arg

* fix: collect properties for cb backend

* fix: collect properties oxd_host

* feat: backup option for copyFile

* fix: collect properties casa for cb backend

* fix: set_configuration for cb

* feat: function determine_key_gen_path()

* feat: conform script wrt latest changes GluuFederation/scim#18

* feat: Update OpenDJ version

* feat: add jetty version

* feat: adjust template for new field, see GluuFederation/scim#22

* fix: fix oxDeviceData size

* fix: fic consent script api version

* feat: jetty-10 integration

* fix: jetty inifile

* fix: jetty inifile

* feat: jetty version is available with option -a

* fix: arg parser

* fix: spanner passport install

* fix: spanner related issues

* feat: gluu-utils

* fix: check City in TUI (ref: #754)

* fix: updated idp.properties for Shibboleth IDP install

* feat: Casa plugin for Stytch Credentials

* fix: use Final binaries

* feat: remove oxaut-rp installation

* fix: idp.session.slop entry in idp.properties prevents Shib IDP start

* fix: opendj version 4.4.12

* fix: re-enable encoding setup.properties

* fix: prevent casa client vanish

* fix: Fix wrong steps count in consent script

* fix: load setup.properties

* fix: update jetty version

* fix: mem calculation when setup.properties loaded

* fix: set opendj ram constraint

* fix: gluu-radius unable to start

* fix: gluu-radius failed to start due to incorrect user/group

* fix: ownership issue

* fix: radius init.d script

Co-authored-by: Mustafa Baser <mbaser@mail.com>
Co-authored-by: Madhumita <madhu@gluu.org>
Co-authored-by: YuriyZ <yzabrovarniy@gmail.com>
Co-authored-by: Jose <bonustrack310@gmail.com>
Co-authored-by: David <nikdavnik@gmail.com>
Co-authored-by: Ganesh <ganesh.sharma@worldiswelcome.com>
Co-authored-by: Mike Schwartz <mike@gluu.org>
Co-authored-by: Djeumen Rolain <uprightech@gmail.com>
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

No branches or pull requests

2 participants