Commit 4435c65
fix: Conftest can now successfully load files using a file URL (e.g.,
* fix:
Conftest encounters errors on Windows when loading file paths that include drive letters (e.g., `C:/path/to/data.yaml`).
Even when using a file URL (e.g., `file:///C:/path/to/data.yaml`), we still face issues.
With these code changes, Conftest can now successfully load files using a file URL (e.g., `file:///C:/path/to/data.yaml`).
We opted for file URLs instead of paths with drive letters (e.g., `C:/path/to/data.yaml`) because OPA does not support file paths with drive letters. For more details, see [this issue comment](open-policy-agent/opa#6922 (comment)).
Resolves: #979
Signed-off-by: Punith C K <pck@veracode.com>
* fix:
Removing WithProcessAnnotation(true) which is not needed for loading data files
Signed-off-by: Punith C K <punithck@itsupports-Laptop.local>
* fix: Conftest can now successfully load files using a file URL (e.g., file:///C:/path/to/data.yaml) on windows
Removing duplicate code
Signed-off-by: Punith C K <pck@veracode.com>
---------
Signed-off-by: Punith C K <punithck@itsupports-Laptop.local>
Signed-off-by: Punith C K <pck@veracode.com>
Co-authored-by: Punith C K <punithck@itsupports-Laptop.local>file:///C:/path/to/data.yaml) on windows (#999)1 parent 5ae180f commit 4435c65
1 file changed
+8
-8
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
125 | 125 | | |
126 | 126 | | |
127 | 127 | | |
128 | | - | |
129 | | - | |
130 | | - | |
131 | | - | |
| 128 | + | |
132 | 129 | | |
133 | 130 | | |
134 | 131 | | |
135 | 132 | | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | 133 | | |
140 | 134 | | |
141 | | - | |
| 135 | + | |
142 | 136 | | |
143 | 137 | | |
144 | 138 | | |
| |||
147 | 141 | | |
148 | 142 | | |
149 | 143 | | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
150 | 150 | | |
151 | 151 | | |
152 | 152 | | |
| |||
0 commit comments