current uid is a stack

[AndreasMadsen]

MakeCallback is sometimes called synchronously, thus uid can have multiple values in the same synchronous context.

[Fishrock123]

I think this should be fixed transitively via #16?

[AndreasMadsen]

I think they are quite unrelated. The fix is just:

const currState = [root];

function asyncInit(uid, handle, provider, parentUid) {
  // get parent state
  const state = (parentUid === null ? currState[currState.length - 1] : stateMap.get(parentUid));

  // add new state node
  stateMap.set(uid, state.add(handle));
}

function asyncBefore(uid) {
  const state = stateMap.get(uid);

  state.before();
  currState.push(state);
}

function asyncAfter(uid) {
  const state = stateMap.get(uid);

  state.after();
  currState.pop();
}

but I haven't been able to create a test script for it.

[Fishrock123]

Sorry, wrong issue. I think this should be fixed by the actual async_hooks impl? aka nodejs/node#8531

[AndreasMadsen]

Yes, it provided an API for getting the contextual parent.

[AndreasMadsen]

Turns out that the basic http test does cover this after all. Fixed in a88b889