openresty / openresty/rds-csv-nginx-module

add "rds_csv_quote_string" command

Open
#3 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C
Stars
21
Forks
15
Avg merge
1h 18m
Merged PRs (30d)
2

Description

can add one command to control this module to quote the string field or not.
so if using this command and command "rds_csv_field_name_header", we can server binary data from RDS upstream (eg: image).

location ~ '/cats/image/([0-9]+)' {
    drizzle_query 'select img from cat_image where id=$1 limit 1';
    drizzle_pass backend;
    rds_csv on;
    rds_csv_quote_string off;
    rds_csv_field_name_header off;
    rds_csv_content_type 'image/jpeg';
}

i have write simple code on the fork https://github.com/lloydzhou/rds-csv-nginx-module/commit/5c2649352fc20d24b26604701118ec7673408877.

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 with the fork commit linked in the issue and the existing rds_csv and rds_csv_field_name_header directives in this module. Verify that rds_csv_quote_string off with the field-name header disabled can return binary RDS data, and confirm the example's image content type behavior before adding coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
c, nginx
Domain
backend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.