openresty / openresty/rds-csv-nginx-module
add "rds_csv_quote_string" command
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
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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