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

softconf2aclpub ISSUES login #186

Open
MirunaClinciu opened this issue Sep 2, 2024 · 1 comment
Open

softconf2aclpub ISSUES login #186

MirunaClinciu opened this issue Sep 2, 2024 · 1 comment

Comments

@MirunaClinciu
Copy link

MirunaClinciu commented Sep 2, 2024

URL Format Issue

When working with the softconf2aclpub.py script, you may encounter issues where the configuration file (config.json) does not align with the actual URL format required by the SOFTCONF platform. For instance, if you are working with the INLG2024 conference, the configuration might initially look like this, based on the README instructions:

error: tconf\softconf2aclpub.py", line 118, in get_conference_details
response = br.open(CONF_URL)

In the README FILE according to instructions

"conf" : "inlg2024",
"track" : "GenerationChallenges"

However, this format might not work as expected. The correct configuration should be:

"conf" : "n/inlg2024",
"track" : "track/GenerationChallenges"

Modifications to Address URL and Login Issues

I had to modified CONF_URL (lines 26-29)

CONF_URL = f"{SOFTCONF_URL}{config['conf']}/{config['track']}/"
Modified to:
CONF_URL = f"{SOFTCONF_URL}{config['conf']}/login/scmd.cgi?scmd=login"

Adjusting the Login Process: The original login handling was unreliable. It attempted to select and fill in the form fields in a way that didn’t always work:

Original code
br.form = list(br.forms())[0]
I replaced this with a more robust method:

br.select_form(nr=0)
br.form["username"] = config["username"]
br.form["password"] = config["password"]
response = br.submit()

@crux82
Copy link
Collaborator

crux82 commented Sep 2, 2024

@MirunaClinciu Thank you for spotting this problem. Can you send a pull request with this specific update? So I can merge it into the main branch!

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