Skip to content

Commit 36046d6

Browse files
committed
Clarifying comments
1 parent 347108e commit 36046d6

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/WebJobs.Script/Description/ScriptFunctionInvokerBase.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,12 @@ protected static Dictionary<string, string> GetBindingData(object value, IBinder
107107
}
108108

109109
/// <summary>
110-
/// TEMP HACK
111110
/// We need to merge the ambient binding data that already exists in the IBinder
112111
/// with our binding data. We have to do this rather than relying solely on
113112
/// IBinder.BindAsync because we need to include any POCO values we get from parsing
114113
/// JSON bodies, etc.
114+
/// TEMP: We might find a better way to do this in the future, perhaps via core
115+
/// SDK changes.
115116
/// </summary>
116117
protected static void ApplyAmbientBindingData(IBinder binder, IDictionary<string, string> bindingData)
117118
{
@@ -135,7 +136,7 @@ private static IDictionary<string, object> GetAmbientBindingData(IBinder binder)
135136

136137
try
137138
{
138-
// TEMP: Dig the ambient binding data out of the binder
139+
// TEMP HACK: Dig the ambient binding data out of the binder
139140
FieldInfo fieldInfo = binder.GetType().GetField("_bindingSource", BindingFlags.NonPublic | BindingFlags.Instance);
140141
var bindingSource = fieldInfo.GetValue(binder);
141142
PropertyInfo propertyInfo = bindingSource.GetType().GetProperty("AmbientBindingContext");

0 commit comments

Comments
 (0)