feat(integrations): add support for avanan#5553
feat(integrations): add support for avanan#5553hassan254-prog wants to merge 4 commits intomasterfrom
Conversation
![propel-code-bot[bot]](https://avatars.githubusercontent.com/in/1179004?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
Review found no issues; changes look solid and aligned with requirements.
Status: No Issues Found | Risk: Low
Review Details
📁 14 files reviewed | 💬 0 comments
Instruction Files
├── .claude/
│ ├── agents/
│ │ └── nango-docs-migrator.md
│ └── skills/
│ ├── agent-builder-skill/
│ │ ├── EXAMPLES.md
│ │ └── SKILL.md
│ ├── creating-integration-docs/
│ │ └── SKILL.md
│ └── creating-skills-skill/
│ └── SKILL.md
├── AGENTS.md
└── GEMINI.md
| x-av-app-id: ${credentials.appId} | ||
| x-av-token: ${accessToken} | ||
| x-av-req-id: ${random} | ||
| x-av-date: '${now | date: "%Y-%m-%dT%H:%M:%S"}' |
There was a problem hiding this comment.
This will allow for specific date format interpolation, avanan doesn’t accept dates in zulu format.
There was a problem hiding this comment.
is that confirmed. the docs I found only have example with zulu format
There was a problem hiding this comment.
Yes, this is confirmed and tested, if you use a zulu format date, this is the error you get.
|
|
||
| // Custom headers handling | ||
| if ('proxy' in config.provider && 'headers' in config.provider.proxy) { | ||
| const headerValues = Object.values(config.provider.proxy.headers).filter((v): v is string => typeof v === 'string'); |
There was a problem hiding this comment.
this will ensure that the dates and random numbers we generate remain consistent across all headers.
| x-av-app-id: ${credentials.appId} | ||
| x-av-token: ${accessToken} | ||
| x-av-req-id: ${random} | ||
| x-av-date: '${now | date: "%Y-%m-%dT%H:%M:%S"}' |
There was a problem hiding this comment.
is that confirmed. the docs I found only have example with zulu format
2 participants
Describe the problem and your solution
Add
avananintegration with enhanced interpolation and proxy header handlingThis PR introduces a new
avananintegration usingTWO_STEPauth, including provider configuration, token/proxy headers, connection and credential schemas, documentation pages, snippets, and a logo asset. It also expands interpolation utilities to supportsha256Hex, formattednow, stable per-requestrandom/now, and `` replacements, with proxy header construction updated to use stable values forTWO_STEPflows.Additional changes include new
formatDatehelper logic, expanded unit test coverage for interpolation and proxy utilities, and a guard in credentials refresh to ensurenewCredentials.rawis an object before extracting tokens.Key Changes
• Added
avananprovider configuration inpackages/providers/providers.yamlwithTWO_STEPauth, token headers, proxy headers, and connection/credential schema• Implemented interpolation enhancements in
packages/shared/lib/utils/utils.ts, includingsha256Hex, formattednow, stablerandom/now, `` replacement, andformatDate• Updated proxy header interpolation for `TWO_STEP` in `packages/shared/lib/services/proxy/utils.ts` to reuse stable `random`/`now` per request
• Expanded test coverage in `packages/shared/lib/utils/utils.unit.test.ts` and `packages/shared/lib/services/proxy/utils.unit.test.ts`
• Added Avanan docs and snippets in `docs/api-integrations/avanan.mdx`, `docs/api-integrations/avanan/connect.mdx`, `docs/snippets/generated/avanan/PreBuiltTooling.mdx`, and `docs/snippets/generated/avanan/PreBuiltUseCases.mdx`, plus navigation updates in `docs/docs.json`
• Added `newCredentials.raw` object guard in `packages/shared/lib/services/connections/credentials/refresh.ts`
Possible Issues
•
token_request_method: GETwith custom headers may require provider validation for header acceptance• Formatted
nowuses UTC formatting; confirm Avanan expects this exact format (no timezone suffix)• Interpolation order for
sha256Hexand nested${...}placeholders should match Avanan signature requirementsThis summary was automatically generated by @propel-code-bot