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

Update Deprecated readfp() Usage #5688

Closed
wants to merge 1 commit into from

Conversation

itsmessk
Copy link

@itsmessk itsmessk commented Apr 12, 2024

"you have provided an invalid and/or unreadable configuration file ('AttributeError: 'UnicodeRawConfigParser' object has no attribute 'readfp'')"

Replaced deprecated readfp() method with read_file() method for Python 3 compatibility in one instance.

Change Made:
Updated readfp() to read_file() in UnicodeRawConfigParser initialization.

@stamparm
Copy link
Member

sqlmap is python2 backwards-compatible too. you can't introduce stuff like this

@stamparm stamparm closed this Apr 12, 2024
stamparm added a commit that referenced this pull request Apr 12, 2024
@stamparm stamparm self-assigned this Apr 12, 2024
@stamparm
Copy link
Member

proper fix with 1e9e33d

googlesky pushed a commit to googlesky/sqlmap that referenced this pull request Apr 14, 2024
googlesky pushed a commit to googlesky/sqlmap that referenced this pull request Apr 14, 2024
* Patch related to the sqlmapproject#4137

* Patch related to the sqlmapproject#5669

* fixed: sqlite3.OperationalError: table logs already exists (sqlmapproject#5677)

CREATE TABLE IF NOT EXISTS

* Removing some obsolete code

* Minor patch

* chore: remove repetitive words (sqlmapproject#5687)

Signed-off-by: laterlaugh <manziwenzhai@sina.cn>

* Trivial update

* Proper patch for sqlmapproject#5688

* Patching some resource-related warnings

---------

Signed-off-by: laterlaugh <manziwenzhai@sina.cn>
Co-authored-by: Miroslav Stampar <miroslav.stampar@gmail.com>
Co-authored-by: G3G4X5X6 <87740076+G3G4X5X6@users.noreply.github.com>
Co-authored-by: laterlaugh <166613655+laterlaugh@users.noreply.github.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

Successfully merging this pull request may close these issues.

None yet

2 participants