datopian / datopian/aircan

Log integration polishing

Open
#90 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
34
Forks
8
Avg merge
2d 22h
Merged PRs (30d)
1

Description

Acceptance

  • Handle large log entries (prevent timeouts)
  • Handle proper encoding
  • Integration test (log consumption)

Tasks

  • Set up a max page size to prevent timeouts (this issue won't implement full pagination, but at least the request won't fail when fetching the logs)
  • Encode JSON response before calling response.json() on aircan-connector

Analysis

PR datopian/ckanext-aircan#21 was merged and it breaks the filter

Notes

These two features are left out
dynamic filters
Customizable filters

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the log integration request that fetches log entries and the aircan-connector response handling mentioned in the issue. Add or run the integration test for log consumption, then verify that a maximum page size prevents timeouts and the remaining acceptance items pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.