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

Bulk update: Fixing session PUT (in handler,y), EL v8 compatibility, python 3 based with enhanced code debugging. #131

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gmkll
Copy link

@gmkll gmkll commented Nov 14, 2024

Refactor: In handler.py method traceback and use in exception; Add print statements (as debug hints) in session,py and other modules; Fix in handler.py by removing bucket (which is no key, value pair) from setting into session headers as argument for start_response call); Add __init__py in plugins folder to allow import from makeAccount.py and remove local KibbleDatabase class there;
Make in database.py *Wrapper classes private;
Add into kibble.yaml configuration key elasticsearch versionHint (by default 8) to allow for more robust and transparent initialization; Refactor in setup.py into methods makeIndices and makeAccount(); Fix check for undefined snap in kibble.v1.js.
Improvement: Evaluate result in makeAccount.py;
Remove cgi import (not used) and deprecated since py 3.11; Use debug to print out more debugging information; Set cookie httpOnly;
Update setup docs (tested with python 3.9.1 and 3.13.0).

Refactor: In handler.py method traceback and use in exception;
Add print statements (as debug hints) in session,py and other modules;
Fix in handler.py by removing bucket (which is no key, value
pair) from setting into session headers as argument for start_response call);
Add __init__py in plugins folder to allow import from makeAccount.py and
remove local KibbleDatabase class there;
Make in database.py *Wrapper classes private;
Add into kibble.yaml configuration key elasticsearch versionHint
(by default 8) to allow for more robust and transparent initialization;
Refactor in setup.py into methods makeIndices and makeAccount();
Fix check for undefined snap in kibble.v1.js.
Improvement: Evaluate result in makeAccount.py;
Remove cgi import (not used) and deprecated since py 3.11;
Use __debug__ to print out more debugging information;
Set cookie httpOnly;
Update setup docs (tested with python 3.9.1 and 3.13.0).
@sharanf sharanf self-assigned this Nov 14, 2024
@gmkll
Copy link
Author

gmkll commented Nov 14, 2024

While this is working for the 1st level integration, more changes are required to support EL8 e.g. this error occurs when data points are selected in kibble

[date_histogram] unknown field [interval] did you mean [fixed_interval]

It might be an idea to create an branch kibbl1-el8 to continue work on this, as this may take some time ..

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.

2 participants