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

Setup fails on Ubuntu 18.04 #152

Closed
1 of 3 tasks
mgmax opened this issue Aug 20, 2018 · 6 comments
Closed
1 of 3 tasks

Setup fails on Ubuntu 18.04 #152

mgmax opened this issue Aug 20, 2018 · 6 comments

Comments

@mgmax
Copy link
Member

mgmax commented Aug 20, 2018

Running the Vagrant setup with "ubuntu/bionic64" causes this error: pyca/cryptography#4020

Best guess: Recent versions of python-cryptography (used by requests) are incompatible with the enum library, but wants the newer enum34 which has a slightly different interface. This is a problem because FAUCardPayment uses enum.

  • rewrite FAUCardPayment to use enum34 (and test it thoroughly!)
  • change requirements.txt
  • update Vagrantfile to "ubuntu/bionic64" and test it
@BDrescher
Copy link
Contributor

BDrescher commented Aug 20, 2018 via email

@BDrescher
Copy link
Contributor

Hey,
I am replying once again as I know wether a edited comment will trigger a notification. For my reply see the message above.

@mgmax mgmax changed the title Crashes on Ubuntu 18.04 Setup fails on Ubuntu 18.04 Oct 5, 2018
@mgmax
Copy link
Member Author

mgmax commented Oct 5, 2018

The vagrant box definitely exists: https://app.vagrantup.com/ubuntu/boxes/bionic64
I created the fix-152-test branch which uses enum34 and ubuntu/bionic64, it does not crash. Because I don't have the real payment hardware, I get a connection error message as expected. I don't know how to test it further without real hardware.

@BDrescher
Copy link
Contributor

I do not doubt it exists, yet it failed to run with me having little knowledge on vagrant. As soon as I have Access and time to the hardware, I will test the branch.

@mgmax
Copy link
Member Author

mgmax commented Oct 6, 2019

I found out that we've been using enum34 on the production system all the time, and the code would never have worked with the old enum. So the FAUCard code does not need to be changed, only requirements.txt was wrong.

mgmax added a commit that referenced this issue Oct 6, 2019
and switch to Debian 10 (Buster) as Vagrant OS
and some minor improvements so that `vagrant up` just works nicely.

Fixes #152
mgmax added a commit that referenced this issue Oct 6, 2019
and switch to Debian 10 (Buster) as Vagrant OS
and some minor improvements so that `vagrant up` just works nicely.

Fixes #152
@mgmax
Copy link
Member Author

mgmax commented Mar 15, 2022

should be fixed

@mgmax mgmax closed this as completed Mar 15, 2022
mgmax added a commit that referenced this issue Mar 22, 2022
and switch to Debian 10 (Buster) as Vagrant OS
and some minor improvements so that `vagrant up` just works nicely.

Fixes #152
patkan pushed a commit that referenced this issue Jun 14, 2022
and switch to Debian 10 (Buster) as Vagrant OS
and some minor improvements so that `vagrant up` just works nicely.

Fixes #152
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