Skip to content
This repository has been archived by the owner on Nov 2, 2023. It is now read-only.

sdk/middleware/sqiris: middleware function for Iris #22

Closed
wants to merge 1 commit into from

Conversation

Julio-Guerra
Copy link
Collaborator

@Julio-Guerra Julio-Guerra commented Feb 2, 2019

Closes SQR-5407

- Middleware function for Iris.
- Iris-specific `FromContext()` function as Iris' context does not implement the
  Go context interface, and does not either allow to modify the request, so it
  is not possible to use `sdk.FromContext()`.
@codecov
Copy link

codecov bot commented Feb 2, 2019

Codecov Report

Merging #22 into dev will decrease coverage by 0.16%.
The diff coverage is 84.61%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev      #22      +/-   ##
==========================================
- Coverage   13.06%   12.89%   -0.17%     
==========================================
  Files          10       10              
  Lines        2052     2048       -4     
==========================================
- Hits          268      264       -4     
  Misses       1766     1766              
  Partials       18       18
Impacted Files Coverage Δ
sdk/middleware/sqiris/iris.go 84.61% <84.61%> (ø)
sdk/middleware/sqecho/echo.go

@Julio-Guerra Julio-Guerra self-assigned this Feb 2, 2019
@Julio-Guerra Julio-Guerra modified the milestones: v0.1.0-alpha.6, v0.1.0 Feb 2, 2019
@Julio-Guerra Julio-Guerra added the sdk SDK feature label Feb 13, 2019
@Julio-Guerra Julio-Guerra changed the title [WIP] sdk/middleware/sqiris: middleware function for Iris sdk/middleware/sqiris: middleware function for Iris Feb 20, 2019
@Julio-Guerra Julio-Guerra added the enhancement New feature or request label May 7, 2019
@Julio-Guerra
Copy link
Collaborator Author

Dropped until we have user request for Iris

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request sdk SDK feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant