Shell commands utility doesn't work for Windows endpoint

I try to use the "run shell command" utility to a Windows server, which has cygwin and openssh in place. It seems that only the first command can run, and all the rest return 255. E.g.
2018-05-29T02:33:28.7209130Z ==============================================================================
2018-05-29T02:33:28.7209595Z Task         : SSH
2018-05-29T02:33:28.7209794Z Description  : Run shell commands or a script on a remote machine using SSH
2018-05-29T02:33:28.7210006Z Version      : 0.121.0
2018-05-29T02:33:28.7210176Z Author       : Microsoft Corporation
2018-05-29T02:33:28.7210397Z Help         : [More Information](http://go.microsoft.com/fwlink/?LinkId=821892)
2018-05-29T02:33:28.7210621Z ==============================================================================
2018-05-29T02:33:28.9760561Z ##[debug]agent.workFolder=D:\a
2018-05-29T02:33:28.9789737Z ##[debug]loading inputs and endpoints
2018-05-29T02:33:28.9800632Z ##[debug]loading ENDPOINT_AUTH_8b5a9178-d622-43cf-95f7-465081e85edb
2018-05-29T02:33:28.9810938Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_8b5a9178-d622-43cf-95f7-465081e85edb_PASSWORD
2018-05-29T02:33:28.9814420Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_8b5a9178-d622-43cf-95f7-465081e85edb_USERNAME
2018-05-29T02:33:28.9815338Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_SYSTEMVSSCONNECTION_ACCESSTOKEN
2018-05-29T02:33:28.9817096Z ##[debug]loading ENDPOINT_AUTH_SCHEME_8b5a9178-d622-43cf-95f7-465081e85edb
2018-05-29T02:33:28.9819744Z ##[debug]loading ENDPOINT_AUTH_SCHEME_SYSTEMVSSCONNECTION
2018-05-29T02:33:28.9831075Z ##[debug]loading ENDPOINT_AUTH_SYSTEMVSSCONNECTION
2018-05-29T02:33:28.9833822Z ##[debug]loading INPUT_COMMANDS
2018-05-29T02:33:28.9836354Z ##[debug]loading INPUT_FAILONSTDERR
2018-05-29T02:33:28.9837071Z ##[debug]loading INPUT_INLINE
2018-05-29T02:33:28.9838598Z ##[debug]loading INPUT_RUNOPTIONS
2018-05-29T02:33:28.9841309Z ##[debug]loading INPUT_SCRIPTPATH
2018-05-29T02:33:28.9843088Z ##[debug]loading INPUT_SSHENDPOINT
2018-05-29T02:33:28.9852428Z ##[debug]loaded 13
2018-05-29T02:33:29.1600450Z ##[debug]check path : D:\a\_tasks\SSH_91443475-df55-4874-944b-39253b558790\0.121.0\task.json
2018-05-29T02:33:29.1601664Z ##[debug]set resource file to: D:\a\_tasks\SSH_91443475-df55-4874-944b-39253b558790\0.121.0\task.json
2018-05-29T02:33:29.1605309Z ##[debug]system.culture=en-US
2018-05-29T02:33:29.1628188Z ##[debug]sshEndpoint=8b5a9178-d622-43cf-95f7-465081e85edb
2018-05-29T02:33:29.1634616Z ##[debug]8b5a9178-d622-43cf-95f7-465081e85edb auth param username = ***
2018-05-29T02:33:29.1635538Z ##[debug]8b5a9178-d622-43cf-95f7-465081e85edb auth param password = ***
2018-05-29T02:33:29.1637087Z ##[debug]8b5a9178-d622-43cf-95f7-465081e85edb data host = <IP_ADDRESS>
2018-05-29T02:33:29.1637557Z ##[debug]8b5a9178-d622-43cf-95f7-465081e85edb data port = 22
2018-05-29T02:33:29.1637965Z ##[debug]Using username and password for ssh connection.
2018-05-29T02:33:29.1640806Z ##[debug]runOptions=commands
2018-05-29T02:33:29.1642708Z ##[debug]commands=ls
2018-05-29T02:33:29.1642985Z ls
2018-05-29T02:33:29.1643400Z ls
2018-05-29T02:33:29.1643772Z ls
2018-05-29T02:33:29.1646980Z ##[debug]failOnStdErr=true
2018-05-29T02:33:29.1661653Z Trying to setup SSH connection to ***@x.x.x.x:22
2018-05-29T02:33:30.8476663Z Successfully connected.
2018-05-29T02:33:30.8477225Z ##[debug]Running command ls on remote machine.
2018-05-29T02:33:30.8477418Z ls
2018-05-29T02:33:30.8478192Z ##[debug]cmdToRun = ls
2018-05-29T02:33:31.7179995Z file1
2018-05-29T02:33:31.7180333Z file2
2018-05-29T02:33:31.7180502Z file3
2018-05-29T02:33:31.7180714Z file4
2018-05-29T02:33:31.7180881Z file5
2018-05-29T02:33:31.7181064Z file6
2018-05-29T02:33:31.7181170Z
2018-05-29T02:33:31.7344979Z ##[debug]code = 0, signal = undefined
2018-05-29T02:33:31.7345576Z ##[debug]Command ls completed with return code = 0
2018-05-29T02:33:31.7346118Z ##[debug]Running command ls on remote machine.
2018-05-29T02:33:31.7346405Z ls
2018-05-29T02:33:31.7346756Z ##[debug]cmdToRun = ls
2018-05-29T02:33:32.3113263Z ##[debug]code = 255, signal = undefined
2018-05-29T02:33:32.3125019Z ##[debug]task result: Failed
2018-05-29T02:33:32.3180149Z ##[error]Command ls exited with code 255.
2018-05-29T02:33:32.3189443Z ##[debug]Processed: ##vso[task.issue type=error;]Command ls exited with code 255.
2018-05-29T02:33:32.3190634Z ##[debug]Processed: ##vso[task.complete result=Failed;]Command ls exited with code 255.
2018-05-29T02:33:32.3190960Z ##[debug]Closing the SSH client connection.

If I manually run ssh to that Windows machine, it shows no issues. I also try to run the same on other linux end-points and they work well.
Looking at the code:
https://github.com/Microsoft/vsts-tasks/blob/bf4e95053e7894a186b965ff8956c023351669ed/Tasks/SshV0/ssh2helpers.ts#L54
I don't think it's vsts specific, but I'd like to try if anyone here knows of any potential causes/solutions, or any flaws in the packages that you use to run shell commands remotely. Thanks in advance.
P.S. I can of course use other means to deploy. I'm asking mostly due to curiosity.

This issue is stale because it has been open for a year with no activity. Remove the stale label or comment on the issue otherwise this will be closed in 5 days


not an issue, just a thanks

Hi! Just wanted to drop a "thank you" here for putting this repo up :) It was helpful to understand S3-FIFO and SIEVE. I'm not using Go for any projects, but Go is easy to understand the algorithms and easy to translate into other languages. Thank you!!

Thank you for your words 👍
This made me feel good :)


Silence deprecation warning

closes #834

Coverage remained the same at 100.0% when pulling 5b83627bf241c50330592565a447207bb47e8743 on 834-deprecation into acc52148518f07b637827ee19b6db1dd24dc9b93 on main.


feat: add scalar integration (alternative to Redoc)

hey y'all, we had a couple requests to integrate Scalar with Aide 🦀!
We had a lot of fun writing rust + making a super fun theme (check the animation on Ferris between light and dark mode toggle)
some features:

fully customizable themes
offline search (through fuse.js)
integrated API Testing Client
modern look
and more ✨

https://github.com/tamasfe/aide/assets/6176314/1b323a5b-b26a-431a-9d13-6916c9f8f8a2
you can also check it out here https://rust.apidocumentation.com/
i wasn't sure on the best way of integrating (i'm very new to rust) so i'm eager and excited for feedback!
Also I can start writing documentation once feedback is recieved ✨

Thank you for the PR, just revert the deletions in the example if there was no reason for it.
@tamasfe do you approve ?
I'll do the merge and bump.

@Wicpar I added the deletions cause of this issue #103 , we need to handle this error in Scalar but I haven't had a moment to roll out the fix.
I deleted it since it doesnt add properly to the schemas and it causes an invalid swagger file as output (can see the thread on missing AppError schema)
Happy to remove but since it doesnt get generated we may want to remove it? Happy on whichever decision y'all think is best here ✨

Yes i posted the details. We can push a fix before merging this PR if you want :)

Yes i posted the details. We can push a fix before merging this PR if you want :)

ah right, sorry haha I had way too much coffee today 👀
i'll remove the deleted code and i'm happy to merge it with a fast follow ❤

Yes i posted the details. We can push a fix before merging this PR if you want :)

ah right, sorry haha I had way too much coffee today 👀
i'll remove the deleted code and i'm happy to merge it with a fast follow ❤

And i need to sleep, i was wrong on the example thing :)

haha we can chat tomorrow @Wicpar ✨ 💤
happy either way on the decision. I reverted the commented out code :)

published in 0.13.2

@marclave Hi, i was wondering why the response schemas are not documented like the request bodies ? Any way to fix than ? other than that the interface has been great.

@marclave Hi, i was wondering why the response schemas are not documented like the request bodies ? Any way to fix than ? other than that the interface has been great.

hi @Wicpar , appreciate the kind words! the response schemas should be rendering, would you be able to provide an example OpenAPI spec and I can investigate ✨

You can look at api.v2.ipocamp.io for my production api
But i checked and it doesn't work in your demo using the swagger pet shop either. The responses only have the example, not the interface to explore the schema like the request body has.


update dependencies

Apparently graph-tool-base as dependecy raises cairo imports that don't work. Dependencies have been updated to include graph-tool, which should automatically include cairo related stuff

Hi! This is the friendly automated conda-forge-webservice.
It appears you are making a pull request from a branch in your feedstock and not a fork. This procedure will generate a separate build for each push to the branch and is thus not allowed. See our documentation for more details.
Please close this pull request and remake it from a fork of this feedstock.
Have a great day!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Won't merge

Hi! This is the friendly automated conda-forge-webservice.
It appears you are making a pull request from a branch in your feedstock and not a fork. This procedure will generate a separate build for each push to the branch and is thus not allowed. See our documentation for more details.
Please close this pull request and remake it from a fork of this feedstock.
Have a great day!


Publish first versions of current unpublished packages

At the time of writing, neither esp-ieee802154 nor esp-lp-hal have had releases. Both of these packages provide some meaningful functionality already, so we should determine what work remains to get these ready to release.

[x] New version of esp-wifi-sys published

https://github.com/esp-rs/esp-wifi-sys/pull/473

[x] Update dependencies and documentation for esp-ieee802154

https://github.com/esp-rs/esp-hal/pull/1730

[ ] ~Align public APIs in esp-lp-hal with esp-hal (to the best of our abilities)~
[x] Update dependencies and documentation for esp-lp-hal

https://github.com/esp-rs/esp-hal/pull/1754

[x] Review (and, if needed, update) relevant CHANGELOG.md, README.md, Cargo.toml, etc. files for both packages

For esp-lp-hal, we should probably try to align the APIs to those in esp-hal the best we can; I will look into this. Also probably needs some housekeeping and documentation, which I will also take care of.

I don't think I will get around to updating all the APIs before the initial release, as there's a bit more work to do than anticipated, so this will likely be left for a subsequent version.

With #1754, esp-lp-hal is mostly in shape for release. I still need to finish updating the I2C module, however I will not consider this a blocker for release. I will try to complete it in time regardless, though. We also need to provide our own APIs instead of relying on embedded-hal and friends, but that can be punted to the next version I think.

Both packages have been published:

https://crates.io/crates/esp-ieee802154/0.1.0
https://crates.io/crates/esp-lp-hal/0.1.0


[Documentation] Update Discord FAQ

I'm filing under bug, since I'm limited to only filing bugs and feature requests.
I'd add a new question:
Q: Messages being sent to the wrong channel
A: You're using webhooks; and the webhook editing functionality is currently broken (#1296). You should move the webhook to the right channel, and make a webhook per channel.
And remove the following question:
Q: Messages not being edited, but a new message with (edit) appears?
Message editing is not supported by discord using webhooks. If you want message editing, disable webhooks and use token only
Since message editing with webhooks is supported and works fine, this should be removed.
I can submit these changes as pull request, if they're okay as in this issue or there's a suggestion to rephrase it.

https://github.com/42wim/matterbridge/wiki/Section-Discord-(basic) should be updated as well since webhook editing doesn't work even if the bot has the required permissions.


Marker is unsupported

Any plans for supporting the Marker element?

It is fourth in the TODO list , at least I don't know anyone working on it, feel free to start implementing :D
https://github.com/react-native-community/react-native-svg#todo

Hey guys, any news on this?

@pedromrb Please feel free to work on it if you have a need for it.

I've made a first attempt at implementing marker element support now.
Could you help testing it? https://github.com/react-native-community/react-native-svg/pull/1122
It's available from the develop branch: https://github.com/react-native-community/react-native-svg/tree/develop

:tada: This issue has been resolved in version 9.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@msand
How can I use it with Path component?
in React we can use it like this:
<Svg>
        <Defs>
          <Marker id={id} markerWidth="13" markerHeight="13" refX="2" refY="6" orient="auto">
            <Path d="M2,1 L2,10 L10,6 L2,2" fill={color}/>
          </Marker>
        </Defs>

        <Path d={line} style={{markerMid: `url(#${id})`}}/>
        <Path key={`transparent ${id}`} d={transparentLine}/>
</Svg>

Is there any correspond prop to marker(Start/Mid/End) in react-native-svg?

@msand
How can I use it with Path component?
in React we can use it like this:
<Svg>
        <Defs>
          <Marker id={id} markerWidth="13" markerHeight="13" refX="2" refY="6" orient="auto">
            <Path d="M2,1 L2,10 L10,6 L2,2" fill={color}/>
          </Marker>
        </Defs>

        <Path d={line} style={{markerMid: `url(#${id})`}}/>
        <Path key={`transparent ${id}`} d={transparentLine}/>
</Svg>

Is there any correspond prop to marker(Start/Mid/End) in react-native-svg?


feat: support animate function

support feature https://github.com/Tahul/pinceau/issues/44

How is this PR doing? I would really like to use this.

The @Tahul  are currently refactoring pinceau, so we may have to wait a while

Hey :)
If you are up to discover the new project architecture and maybe port this PR to the new repository, I would be happy to review it and maybe merge it!
Sorry for the big mess I made with #95

Don't be sorry, this is an prettier way to make the pinceau prettier. I'm formatting the pinceau Prettier at the moment, because I find that the Prettier doesn't work for functions on


Tests for CA, CG and TPR classes

Fixes #157

Perfect. Just rebase from the most recent master and we are ready to merge.


WIP: Compatibility with upcoming sbuild support in Isar

This patchset makes meta-iot2050 compatible with upcoming sbuild support in Isar.
There are 5 patches here.
First 3 of them can be used without sbuild on current 'next':

tcf-agent: fix debian/rules permissions
npm: Add libnode-dev build depends to some npm packages
u-boot-iot2050: Fix building with sbuild

The following patch is related to sbuild, but should also be 'next'-compatible:

sbuild: npm: introduce and use npm_do_mounts and npm_undo_mounts

And the last patch uses sbuild-specific exports, so it is not compatible with 'next'

sbuild: Use sbuild-export API for meta-iot2050 packages

The patchset is tested with the following changes in kas-container:
diff --git a/kas-container b/kas-container
index 787f65b..7277196 100755
--- a/kas-container
+++ b/kas-container
@@ -315,9 +315,13 @@ if [ "$(id -u)" -eq 0 ] && [ "${KAS_ALLOW_ROOT}" != "yes" ] ; then
        exit 1
 fi

+# We need to use external dirs for schroot overlay
+trace mkdir -p "${KAS_BUILD_DIR}/schroot_overlay"
+
 set -- "$@" -v "${KAS_REPO_DIR}":/repo:${KAS_REPO_MOUNT_OPT} \
        -v "${KAS_WORK_DIR}":/work:rw --workdir=/work \
        -v "${KAS_BUILD_DIR}":/build:rw \
+       -v "${KAS_BUILD_DIR}/schroot_overlay":/var/lib/schroot/union/overlay:rw \
        -e KAS_BUILD_DIR=/build \
        -e USER_ID="$(id -u)" -e GROUP_ID="$(id -g)" --rm

diff --git a/kas/iot2050.yml b/kas/iot2050.yml
index 901b164..1e8da00 100644
--- a/kas/iot2050.yml
+++ b/kas/iot2050.yml
@@ -27,12 +27,9 @@ repos:

   isar:
     url: https://github.com/ilbers/isar
-    refspec: e274130b870b7e31532fe191b9932cde1d819b4b
+    refspec: 37a71e72c7b8e9ae7e9f6a402ecdc6ff66eab8c4
     layers:
       meta:
-    patches:
-      u-boot-fix:
-        path: isar-patches/0001-u-boot-Exclude-libubootenv.a-from-dh_strip.patch

 bblayers_conf_header:
   standard: |

Also, modified docker image currently should be used:
export KAS_CONTAINER_IMAGE_PATH=ghcr.io/wiselord
export KAS_IMAGE_VERSION=2.6.2-sbuild

Marked it "WIP" / "Draft" to clarify that this is not yet merge material but shall help with the Isar interface discussion.


overlap analysis & evaluate phases

to graphically show there intense relationship...

wont fix-  let's keep the phases separate.


EKS Cluster Setup Documentation

Add prerequisites to setting up KubeArmor in a EKS Cluster in the getting started  guide as a section
Reference https://github.com/kubearmor/KubeArmor/tree/main/deployments/EKS#prerequisite-for-the-deployment
Hi, If it is the first time that you contribute to KubeArmor, follow these steps:
Write a comment in this issue thread to let other possible contributors know that you are working on this bug. For eg : Hey all, I would like to work on this issue., checkout Contributing Guide :fire::sparkles: and feel free to ask anything related to this issue in this thread or on our slack channel ✌🏽

@daemon1024 , I think we have the necessary documentation in the context. What is the ask here?

Yup. Closing this. Need to clean up list in https://github.com/kubearmor/KubeArmor/issues/477


Use generic kernel on P6 & P7 so that lapack-test passes

When we run lapack-test on Power7, we found some Eigen test failed. This PR tries to turn back to generic kernel for GEMV and GEMM, etc. Then all test passed on Power7 AIX.

LGTM

Hmm, I'm not entirely sure this is necessary. Which compiler did you use ?

We use IBM OpenXL C/C++ 17.1.2 and OpenXL Fortran 17.1.2 on AIX. No failure.
I also tried GCC 11.3.1 on RHEL9.2. Only 1 failure in ssvd.out (SBD, M=30, N=40, type=10, test(9) returns 50.50 and thresh is 50.00).
Without the change, there are about 205 (AIX7.2, OpenXL), 189 (RHEL9.2, GCC) failures.

Thanks - the question for me is why this comes up only know, and if the errors are of any significance. The fundamental problem with the LAPACK testsuite is that it expects the exact same results as would be obtained from the unoptimized Reference LAPACK implementation compiled with the equally unoptimized Reference BLAS, and any hint of FMA operations or just a different order of summations can throw it off.

Yes, we started to look at the lapack-test failures on Power platform. We worked on Power7 firstly because that is old and may not impact too much.
You are correct that these test expect exact same result from the LAPACK subroutines which is something like bit accurate. The current failures are caused by not only FMA operation but also something else. What I found is that the Complex GEMM implementation in kernel/power/zgemm_kernel_power6.S computes the complex multiply and add by computing all RR and II seperately. Then in the final step, it sub the sum of II from the sum of RR. And the similar idea in Imag part by adding sum of RI and sum of IR.
By switching to the generic C implementation, all test passes except 1 failure in ssvd.out (Linux).

As @RajalakshmiSR suggested, I restored the Makefile.power for P6. Now only change the KERNEL for Power6.

Can we target this for 0.3.28?

@martin-frbg Can we target this for 0.3.28? Thanks.

@RajalakshmiSR I'm not particularly happy with throwing out the GotoBLAS kernels when this are probably "minor errors" in the terminology of the netlib LAPACK FAQ. On the other hand if I misunderstood and the errors are seen with the testsuite from Eigen ...

@martin-frbg Sorry for the confusion. The lapack testing on Power7 may not be "minor failure" because one of the failure is about 0.839E+07. I think some of the EigenValue outputs are not in the same order.
Then how about change in GEMM or GEMV assembly kernels? After looking into the assembly code, we found of the code change the computation order. If this is a preferred way to fix the LAPACK testing failures, I can close this PR and create another PR for that change.
Thank you for your review and comments.

Close this PR and rework on GEMV/GEMM assembly kernels.


Fix Stryker runner failure

Although unit tests are working fine, the Stryker runner is getting an error every time. This started after we updated the Stryker dependencies versions. We need to check if we can solve this within the newer versions or go back to the older versions.

Fixed with stryker libraries update.


Limit of 100 stickers

Hi, im not sure if its an issue, but seems like there is a limit on 100 images ¿Can do this limit bigger?
Thanks!!

Yes, it's 100 now.
And I am considering to upgrade it to 200.


Login fails

I get "Error! There is no OAuth secret!" on login attempt.
Firefox 85.0.1

That looks like a storage issue. Even if you are still logged in, editing doesn't work because there is no data about the objects stored ("no base data" or similar, I don't know the precise English error message)

Thanks @mueschel, it really was. Try again Mateusz, please.

For me both logging in and editing works again. Thanks!

Thanks, now it works!

@Zverik
Seems the issue is back.
First I got No base data for ... while still logged in.
After logging out I can't log in anymore because of Error! There is no OAuth secret!

I can confirm that the same symptoms are back.

Could relogin in the meantime.
Maybe a transient storage issue? If the .base files fill up the available space faster then some cron job cleans them out

Maybe a transient storage issue?

Same problem is back again.

Same problem is back again.

Again the free disk space issue — fixed it for good, hopefully. Thanks for the notice!

Same problem is back again.

Fixed. Sorry :)


ChatAttachment class improvement suggestion

current code
export class ChatAttachment {
  readAttachment(json:any):void

how to use
let a = new ChatAttachment();
a.readAttachment(JSON);
chat.replyTemplate(new AttachmentTemplate(a));

if readAttachment function returns 'this' or ChatAttachment class has 'static builder(factory)', it can be used like this:
chat.replyTemplate(new AttachmentTemplate(new ChatAttachment().readAttachment(JSON)));

or
chat.replyTemplate(new AttachmentTemplate(ChatAttachment.BuildAttachment(JSON)));

There are planned improvements for attachments. Was supposed to be updated already. But delayed due to low priority for enhancements. The problem is that nested objects are too messy.
Current design
new CustomAttachment(new CustomInfo(/*very long arguments*/), content);

Inner contents(AttachmentContent) can be improved like this
new CustomAttachment({ message: 'link', type: CustomType.FEED, /* e.t.c. */ }, { /* contents */ });

This suggestion is unsafe because attachment count, type are determined by chat type.
chat.replyTemplate(ChatAttachment.fromJson(JSON).template);


[Bug]: click events not working on table rows inside GoATable

Describe the bug
If you use GoATable in React. And attach an onClick event to a table row inside the table, the click event will not fire.
Provide Code
function consoleLog() {
  console.log("I'm in the table row");
}

<GoATable width="100%">

      First Name
      Last Name
      Age

 consoleLog()}>
      Dustin
      Nielsen
      37

      This is the footer
      This is the footer
      This is the footer

</GoATable>

To Reproduce
Use the above code, click the row, a console message should show up. It's not.
Relevant console log output
No response
What browsers are you seeing the problem on?
No response
Additional context
From @chrisolsen "It looks like the table is swallowing all the events within it, including clicks".

:tada: This issue has been resolved in version 4.0.0-alpha.176 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0-alpha.213 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.4.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.13.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.5.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.13.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.13.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.13.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


update wizard variations to have consistent styles

Wizards were using different greens and didn't consistently indicate the active and completed steps.

I believe this had to do with inconsistency with legacy software using the uikit?
I think the design had the border and the fill colors to be the same as discussed in:  https://github.com/Availity/availity-uikit/pull/51
Nice! seemed to fix the active text font weights :+1: I was having issues with it not taking.

@bobbennett is this ready for merge? if so please go ahead.


Suggestion: Easy On Boarding Flow & Documentation

I'm going to try to run this within Cloud9.
I got this much to display:

This is the error I'm getting:
Executing (default): SELECT "id", "Geometry", "Fund_St", "Legacy_ID", "Lead_Ag", "Proj_Title", "Proj_Ty", "Proj_Desc", "Contact_info", "More_info", "Primary_Street", "Cross_Streets", "Proj_Status", "Proj_Man", "CD", "Access", "Dept_Proj_ID", "Other_ID", "Total_bgt", "Grant", "Other_funds", "Prop_c", "Measure_r", "Gas_Tax", "General_fund", "Authorization", "Issues", "Deobligation", "Explanation", "Constr_by", "Info_source", "Grant_Cat", "Grant_Cycle", "Est_Cost", "Fund_Rq", "Lc_match", "Flagged", "TotalUnmetFunding", "ProjectStartDate", "ProjectProjectedCompletionDate", "Dup_ID", "Attachment", "createdAt", "updatedAt" FROM "Projects" AS "Project" WHERE "Project"."Access" = 'Public';
Unhandled rejection SequelizeDatabaseError: column "TotalUnmetFunding" does not exist
    at Query.formatError (/home/ubuntu/workspace/node_modules/sequelize/lib/dialects/postgres/query.js:357:14)
    at null.<anonymous> (/home/ubuntu/workspace/node_modules/sequelize/lib/dialects/postgres/query.js:88:19)
    at emitOne (events.js:77:13)
    at emit (events.js:169:7)
    at Query.handleError (/home/ubuntu/workspace/node_modules/pg/lib/query.js:143:8)
    at null.<anonymous> (/home/ubuntu/workspace/node_modules/pg/lib/client.js:180:26)
    at emitOne (events.js:77:13)
    at emit (events.js:169:7)
    at Socket.<anonymous> (/home/ubuntu/workspace/node_modules/pg/lib/connection.js:136:12)
    at emitOne (events.js:77:13)
    at Socket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at Socket.Readable.push (_stream_readable.js:111:10)
    at TCP.onread (net.js:537:20)
From previous event:
    at Query.run (/home/ubuntu/workspace/node_modules/sequelize/lib/dialects/postgres/query.js:74:17)
    at /home/ubuntu/workspace/node_modules/sequelize/lib/sequelize.js:849:20
    at /home/ubuntu/workspace/node_modules/sequelize/node_modules/retry-as-promised/index.js:40:21
From previous event:
    at retryAsPromised (/home/ubuntu/workspace/node_modules/sequelize/node_modules/retry-as-promised/index.js:30:10)
    at /home/ubuntu/workspace/node_modules/sequelize/lib/sequelize.js:848:12
From previous event:
    at Promise.then (/home/ubuntu/workspace/node_modules/sequelize/lib/promise.js:21:17)
    at Model.findAll (/home/ubuntu/workspace/node_modules/sequelize/lib/model.js:1395:6)
    at /home/ubuntu/workspace/controllers/projects_controller.js:56:22
    at Layer.handle [as handle_request] (/home/ubuntu/workspace/node_modules/express/lib/router/layer.js:95:5)
    at next (/home/ubuntu/workspace/node_modules/express/lib/router/route.js:131:13)
    at Route.dispatch (/home/ubuntu/workspace/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/home/ubuntu/workspace/node_modules/express/lib/router/layer.js:95:5)
    at /home/ubuntu/workspace/node_modules/express/lib/router/index.js:277:22
    at Function.process_params (/home/ubuntu/workspace/node_modules/express/lib/router/index.js:330:12)
    at next (/home/ubuntu/workspace/node_modules/express/lib/router/index.js:271:10)
    at Function.handle (/home/ubuntu/workspace/node_modules/express/lib/router/index.js:176:3)
    at router (/home/ubuntu/workspace/node_modules/express/lib/router/index.js:46:12)
    at Layer.handle [as handle_request] (/home/ubuntu/workspace/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/home/ubuntu/workspace/node_modules/express/lib/router/index.js:312:13)
    at /home/ubuntu/workspace/node_modules/express/lib/router/index.js:280:7
    at Function.process_params (/home/ubuntu/workspace/node_modules/express/lib/router/index.js:330:12)
    at next (/home/ubuntu/workspace/node_modules/express/lib/router/index.js:271:10)
    at /home/ubuntu/workspace/node_modules/express/lib/router/index.js:618:15
    at next (/home/ubuntu/workspace/node_modules/express/lib/router/index.js:256:14)

I tracked down the error - the db migration file doesn't contain 3 fields from the Project model: TotalUnmetFunding, ProjectStartDate, and ProjectProjectedCompletionDate.  Thus, columns don't exist for them in the database and we get an error when trying to query.

I think I can help you with that. You just need to delete the database, run the app, so the new columns get added to a not yet existing database and then do the migration one more time. I was part of the team who worked on the latest version of the app :)

Hm, I'm not sure how that would work... if the database doesn't exist when I run npm start it throws a SequelizeConnectionError.
Shouldn't the Project table's migration script define TotalUnmetFunding, ProjectStartDate, and ProjectProjectedCompletionDate?  Those fields are in the model.

You get an error because there are 3 fields in the model that are not in the migration (The ones mentioned above). If you delete the database, and then start the app, Sequelize will create a new database with all the fields in the model (Including TotalUnmetFunding, ProjectStartDate, and ProjectProjectedCompletionDate). Now that the fields are added, then you do the migration on this new database with all the fields and will have no more issues. What you can do next, is add these changes to the migration.
Just wanted to share some insight since we all had to do this at some point this at some point, but forgot to add this to the migration file

As long as you have the right login information in the config file (to connect to your local PostgreSQL) when you start the app, it creates the database

Interesting... I thought I did delete the database (keeping the same connection info as before) but when I ran the app, it threw an error saying there's no database to connect to.  I'll have to do some more research into this to figure out what's going on.  BTW, thanks for taking the time to respond!
Would it break anything if I just add the fields to the migration file?  I'm trying to document an easy on-boarding process for people attending Civic Hack Night.

@matikin9 I do not think that will be an issue.

I want to make a correction. The database does need to exist for Sequelize to connect. That's why you need to specify the database on the config file. What you can do is drop all the existing tables and start the app. Then add the fields to the existing migration file

@crisr10 Sweet, I've made progress.  The issue I'm running into now is when I try to run node load.js.  The issue seems to be that in /models/index.js, the environment variable NODE_ENV is not defined in process.env.NODE_ENV, and thus the config variable is undefined.
I thought that adding these lines to package.json would have set the environment variable:
"scripts": {
    "start": "NODE_ENV=development node ./bin/www"
},

Alrighty - I added instructions for setting the environment variable within Cloud9.  Now my error is:
$ ./transform_data.sh
psql: FATAL:  database "dot" does not exist

Am I supposed to do anything to the ATD_project database.gdb.zip file, which is sitting within \etl\data?

I think this is the issue I'm seeing currently.  Inside the transform_data.sh file, the below line is trying to open a postgresql database named dot:
# remove temp tample
psql -d dot -c "DROP TABLE ogrgeojson"

However, based on following the instructions in the README file so far, these are the only databases I have:
postgres=# \l
                              List of databases
   Name    |  Owner   | Encoding | Collate |  Ctype  |   Access privileges
-----------+----------+----------+---------+---------+-----------------------
 postgres  | postgres | UTF8     | C.UTF-8 | C.UTF-8 |
 template0 | postgres | UTF8     | C.UTF-8 | C.UTF-8 | =c/postgres          +
           |          |          |         |         | postgres=CTc/postgres
 template1 | postgres | UTF8     | C.UTF-8 | C.UTF-8 | =c/postgres          +
           |          |          |         |         | postgres=CTc/postgres
(3 rows)

@matikin9 the transform_data.sh script is only for the "real" data, which isn't open due to privacy concerns.
The data script you want to run and follow is for the db/projects.js file, which has fixtures data on how to load the app. Once you load the fixtures data and create an account (projecturl/users/signup) I think you should be good to go.

@hunterowens I see - so we don't need to do the Loading Data portion of the README?  Perfect :ok_hand:  we're good to go then:

👍

@matikin9 can you review readme changes in last commit and close issue?

Looks good!  I've updated the README on my fork to break down the instructions for using Cloud9: https://github.com/matikin9/dot-planner


Allow user to configure video coding format

User should be able to choose

Could also maybe extend this issue to include VP8 and VP9, whilst they are less common formats they are royalty free and open formats and I feel like CameraX could include support for those later down the line.

As the successor of these formats, I believe that AV1 support should also be considered, but with lower priority as it is currently not widely supported

I second this.
@flawedworld this is an issue tracker, so entries should be titled as such. In this case, I suggest:

Video codec (format) cannot be configured


flowey: support installing deps for cross compiling x64 from aarch64 (ie for ochl)

Title. If you create a devcontainer on aarch64, it doesn't work, needs extra deps which I haven't figured out yet.

Note that we don't actually have a standalone flowey tool that installs system deps for devs. I was hoping we'd lump this into restore-packages, but that ended up getting punted to post oss.

@daprilik let's say I wanted to take a stab at part of this (I'm not saying I will, but I want to have the time to do so). How would I get started?
I just got annoyed at having to do the following on a machine that I couldn't copy/paste to (long story ...):
sudo apt install \
  binutils              \
  build-essential       \
  gcc-aarch64-linux-gnu \
  libssl-dev

@mattkur, I alluded to it above, but I would imagine that we'd want to lump this into the restore-packages job, which would transform into a more generic "set up my dev-env for me please" job.
Installing system deps can be done via the flowey_lib_common::install_dist_pkg node, which accepts a Vec<String> of package names to install. It should be pretty straightforward to weave that dependency into the current job.


Can we use G4 click events to establish the nested level of tree links accessed?

Short answer yes.
After going round the houses a bit I settled on using a custom javascript variable to grab the aria-level from the related list item, which we can then send with the event.
In order to access this data via reports we'd need to set up a custom dimension/metric in GA, which I haven't done as I think this needs wider team discussion.
I've learnt a bit about GA4/GTM from looking into this and I'm going to write this up and share it with the team.


Topics: Add Vala language

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Vala is used by hundreds of open source apps developed for GNOME, elementary OS, and other Linux-based operating systems. The added context of what "Vala" is would be useful when browsing repositories tagged with the topic.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Thanks for the contribution!


MAINTAINERS: Add Jorge Prendes(jprendes) as a REVIEWER

I'd like to invite @jprendes as a reviewer. He had made sustained commits to runwasi, including his work on linking against WasmEdge static library. He has demonstrated his technical depth and knowledge on this project.
Needs explicit LGTM from @jprendes  and 1/3 of the runwasi Committers according to https://github.com/containerd/project/blob/main/GOVERNANCE.md.

[x] @devigned
[ ] @cpuguy83
[ ] @danbugs

This PR will remain open for 7 days.

LGTM! Thanks 😀

LGTM!


Improve Docs

@CmeherGit, once you've signed the CLA we'll be very happy to review this proposed change! Welcome.

/close


Horizontal mouse wheel scrolling?

Not an issue, but just wanted to ask if this plugin supports horizontal scroll with mouse wheel? Or any examples on how to implement this.
Thanks for a great plugin.

Horizontal scrolling is supported in most environments. If you can't do so, it means the input of your mouse wheel is not standard and I am not able to get deltaX value from it.
You can refer to this module to see how it handles wheel event.

@idiotWu Alright, but I am not able to do something like this http://paack.fr/ out of the box?

Ah, I took a look at the minified source code and found something interesting:
void (void 0 === window.SCROLL_DIRECTION || "vertical" === window.SCROLL_DIRECTION ? t.__addMovement(a, l, !0) : t.__addMovement(l, a, !0)))

It seems that the author modified wheel event handler in lib/events/wheel.js.

@jesperlandberg I have the same question, did you resolve with some tricks?
I don't think it's a good idea edit the source code directly.

Nope, By the same reason u didn't. Would be nice to see an option for this tho=)

@jesperlandberg @sndrgb well I think it's VERY weird to perform a horizontal scrolling while I scroll the wheel vertically, I'm afraid there's not going to be an option for that :(

Just thinking about adding a reducer concept which may solve this issue, check #84

Hey there! 8.0.0 is out! Any feedback would be greatly appreciated 🙌 .

Just dropping the solution that helped me do horizontal scroll here for the future :
https://codepen.io/idiotWu/pen/KJNYye


There is a stored xss vulnerability

There is a stored xss vulnerability in the leave reasons in the leave request。
poc：

poc：
<object data="data:text/html;base64,PHNjcmlwdD5hbGVydCgveHNzLyk8L3NjcmlwdD4=">

Thank you for providing the bug. If you have a better plan, you can pr it up.


Update project documentation and scripts

Updates project documentation, licensing, and scripts to enhance usability and maintain legal accuracy.

Updates README.md: Enhances the project's README to include cross-platform compatibility, updates installation and usage instructions, and adds a section for test files. This reflects the latest project changes and broadens the project's accessibility.
Revises LICENSE: Updates the copyright year in the LICENSE file to 2023, ensuring legal accuracy.
Expands .gitignore: Adds entries for additional files and directories, including .vscode/, *.log, and *.tmp, catering to the project's evolving needs.
Populates requirements.txt: Lists all necessary Python packages for the project, such as Django and djangorestframework, facilitating easier setup for new contributors.
Enhances setup_python_env.sh: Converts the script to support additional operating systems beyond macOS, making the project more accessible to a wider audience. The script now prompts for project name and export location, checks for Python3 installation, creates project directories, and opens the project in Visual Studio Code if available.

For more details, open the Copilot Workspace session.

Great


car docs bot: fix diff bot

just testing. should have a unit test for the car docs diff bot in case a structural change is made, it fails silently.

@maxime-desroches any simple test to catch this would be great


Joy Axis in Input Mappin

Operating system or device - Godot version:
Windows 10, Godot 3.0 Alpha Build 1
Issue description:
I tried mapping Joy Axis Device 0 Axis 0 - Left Stick Left to "ui_left"
However it wasn't reading the mapping when using Input.is_action_pressed("ui_left")
Also when I reopened the project the mapping I added was gone
Steps to reproduce:
in Project Settings > Input Mapping > "ui_left" add Joy Axis Device 0 Axis 0 - Left Stick Left
If you reopen the project it won't be mapped anymore.
If you create a script and do if Input.is_action_pressed("ui_left"): print('here')  , "here" won't show up because it doesn't read the Joy Axis has being mapped.
Link to minimal example project:

This was fixed in the master branch in https://github.com/godotengine/godot/commit/5052cb2b9104f57d0ff38d80da50bbeceb926e6f (I know because I reported the bug) and is now working on my machine.
Try using one of the latest builds as this should solve your problem.

Fixed by 5052cb2.

Also OP, just wanted to throw some advice: Keep in mind that this does not take into account "how much" the axis has been moved, only that it has been moved.
This is fine for character movement, and in my project I use "Input.is_action_pressed()" for character movement using left stick, but I use "Input.get_joy_axis(0,2)", which returns the axis value, for controlling the camera with the right stick.
This allows for more sensitivity in camera rotation (which you definitely want, trust me, or else the controls will be physically painful).

I'm looking at dolson's Joystick Project and just trying to build stuff from scratch to understand how it works.   I think he uses Input.get_joy_axis as well.


Add Java goody for traversing a binary tree post-order

Can draw inspiration from the TypeScript goodies in #157 and the Java counterpart in #167.

@miorel can you suggest me which ticket should I pick next? I am ok with either java or typescript.

Do you have some time for a quick call sometime? I think it will be easiest to discuss live. Send me an email (you can find it in git log) and we can plan something.


RenderConfig loading error

[SDL][14.031] Scene objects count: 14
[SDL][14.031] Light definition: LUXCORE_LIGHT_0
[SDL][14.094] Light definition: LUXCORE_LIGHT_1
[LuxCore][14.094] Configuration:
[LuxCore][14.094]   opencl.platform.index = -1
[LuxCore][14.094]   opencl.cpu.use = 0
[LuxCore][14.094]   opencl.gpu.use = 1
[LuxCore][14.094]   renderengine.type = "PATHOCL"
[LuxCore][14.094]   sampler.type = "METROPOLIS"
[LuxCore][14.094]   film.filter.type = "BLACKMANHARRIS"
[LuxCore][14.094]   accelerator.instances.enable = 0
[LuxCore][14.094]   film.width = 1296
[LuxCore][14.094]   film.height = 772
[LuxCore][14.094]   film.imagepipeline.0.type = "TONEMAP_AUTOLINEAR"
[LuxCore][14.094]   film.imagepipeline.1.type = "GAMMA_CORRECTION"
[LuxCore][14.094]   film.imagepipeline.1.value = 2.20000005
[LuxCore][14.094]   batch.halttime = 0
[LuxCore][14.094]   batch.haltspp = 0
[LuxCore][14.094]   path.maxdepth = 10
[LuxCore][14.094]   film.filter.xwidth = 1.64999998
[LuxCore][14.094]   film.filter.ywidth = 1.64999998
[LuxCore][14.094]   sampler.metropolis.maxconsecutivereject = 512
[LuxCore][14.094]   sampler.metropolis.largesteprate = 0.400000006
[LuxCore][14.094]   sampler.metropolis.imagemutationrate = 0.100000001
[LuxCore][14.094] File Name Resolver Configuration:
[LuxCore][14.094]   .
[LuxCore][14.094]   D:/Users/####/Documents/Sketchup
RenderConfig loading error:
Unknown render engine type in RenderEngine::ToProperties(): PATHOCL
When I try to load an lxs file in luxcoreui i got this error. If someone can help ?
You have to know that the first engine i ran was the opencl one, but I don't have openCL.
Thanks in advance.

The default render engine is PATHOCL (i.e. rendering using OpenCL GPUs) while I assume you are using a LuxCore version with CPU-only support.
You should edit the .cfg file and change this line:
renderengine.type = "PATHOCL"
in:
renderengine.type = "PATHCPU"

I'm so glad you answer that fast !
I'm sorry but I don't know which cfg you are talking about ?

Ok I think I understood how the whole things works. Except about this point : why when I write this in the .lxs file :
Renderer "luxcore"
	"string config" ["opencl.gpu.use = 0" "opencl.cpu.use = 1" "renderengine.type = PATHCPU"  "accelerator.type = AUTO"]

...the log still tells me this :
[LuxCore][8.594] Configuration:
[LuxCore][8.594]   opencl.platform.index = -1
[LuxCore][8.594]   opencl.cpu.use = 0
[LuxCore][8.594]   opencl.gpu.use = 1
[LuxCore][8.594]   renderengine.type = "PATHOCL"
[LuxCore][8.594]   sampler.type = "METROPOLIS"
...

Are you using luxcoreui to read the .lxs file ? Just add a "-D renderengine.type PATHCPU" on command line before the .lxs file name.
".lxs" is the old file format used by LuxRender v1.x, it is only partially emulated in v2.x. A new file format is used there.

I'm using the SU2LUX extension, I generated .lxs file with it. I may investigate and see if I find a way to generate .cfg file instead.

As far as I remember, SU2LUX has never been updated to support new file format but the (partial) .lxs parse should be enough for most of the stuff. AS I wrote, you can overwrite any property from the command line, for instance:
luxcoreui -D renderengine.type PATHCPU -D sampler.type SOBOL file.lxs

Thank you so much ! I'm sorry, I don't use ruby so I have some difficulty to guess the correct syntax.
The first time I tried passing the path of my file with -o so it doesn't worked.
Also, can you tell me the correct syntax for absolute link in ruby (in luxecorerender) since apparently, the windows type of link D:\... doesn't work.

I don't know how SU2LUX works, I was under the impression you were saving the scene to a .lxs file and than you were running luxcoreui command by hand.
Now I have the feeling luxcoreui is execute automatically by SU2LUX. Are you using SU2LUX from https://github.com/LuxCoreRender/SketchUp2LuxCore or an older version ?

Yes, I'm using latest SU2LUX to export scene to .lxs and also yes, I'm using luxcoreui to run the lxs render by hand. Apparently, it worked !

Ok, good.


Add LDAP group sync to Teams, fixes #1395

Add setting for a JSON that maps LDAP groups to Org Teams. Format:

{"cn=MyGroup,cn=groups,dc=example,dc=org": {"MyGiteaOrganization": ["MyGiteaTeam1", "MyGiteaTeam2", ...], ...}, ...}

Sync is being run on login and periodically.
Existing group filter settings are reused.
Removal of users from Teams can optionally be enabled.
Teams that are not listed in the mapping JSON are not touched.

fixes #1395

@svenseeberg can you resolve conflicts :)
PS: since its a pull from an org we maintainer cant apply code suggestions or resolve conflicts. if you need help just tell us.

@svenseeberg can you resolve conflicts :)
PS: since its a pull from an org we maintainer cant apply code suggestions or resolve conflicts. if you need help just tell us.

Right, should not be an issue. We will take care of rebasing on the current main branch.

I'll also look into the linting errors.

make fmt

you dont have to hurry we are currently in feature-freeze ... :)
& we need some tests for this code

we need some tests for this code

Are there any specific requirements for the tests? Mocking an LDAP server is somewhat complicated ;-) However, we could easily test the functions that do not interact with the LDAP server.

well we have unit tests who test selve contained functions or easy to moke on ...
example: https://github.com/go-gitea/gitea/blob/main/modules/auth/openid/discovery_cache_test.go
and we have integration tests:
example: https://github.com/go-gitea/gitea/blob/main/integrations/auth_ldap_test.go

I guess we can work with that :-)

We have some LDAP tests already which use a docker container for the LDAP.

Codecov Report

Merging #16299 (89335ed) into main (6554835) will increase coverage by 0.05%.
The diff coverage is 77.03%.

:exclamation: Current head 89335ed differs from pull request most recent head 28ba4d1. Consider uploading reports for the commit 28ba4d1 to get more accurate results

@@            Coverage Diff             @@
##             main   #16299      +/-   ##
==========================================
+ Coverage   45.42%   45.47%   +0.05%
==========================================
  Files         709      709
  Lines       83500    83733     +233
==========================================
+ Hits        37929    38080     +151
- Misses      39490    39553      +63
- Partials     6081     6100      +19

Impacted Files
Coverage Δ

cmd/admin_auth_ldap.go
76.05% <0.00%> (-2.21%)
:arrow_down:

services/forms/auth_form.go
100.00% <ø> (ø)

models/login_source.go
32.61% <64.70%> (+3.97%)
:arrow_up:

modules/auth/ldap/ldap.go
54.71% <88.00%> (+7.89%)
:arrow_up:

models/user.go
52.83% <100.00%> (+1.60%)
:arrow_up:

routers/web/admin/auths.go
35.86% <100.00%> (+0.70%)
:arrow_up:

modules/migrations/github.go
69.77% <0.00%> (-7.41%)
:arrow_down:

models/unit.go
41.09% <0.00%> (-2.74%)
:arrow_down:

modules/migrations/migrate.go
27.20% <0.00%> (-1.58%)
:arrow_down:

models/repo_list.go
77.04% <0.00%> (-0.78%)
:arrow_down:

... and 20 more

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 6554835...28ba4d1. Read the comment docs.

@6543 we updated our pull request an included tests. Can you please review?

@svenseeberg can you resolve conflicts :)

@6543 Thanks a lot for your review
The merge conflicts are now resolved
Appreciate your feedback

I'm worried that this might be too complex.
gohabor has a much simpler way of assigning groups and we could simply say our groupnames have to be organization:team

I'm worried that this might be too complex.

From my point of view this is not more complicated than LDAP filters. Writing a mapping JSON should be possible. However it would be nice to have some sort of syntax validation. But there are plenty of tools that can help with that. I think that admins tending to LDAP servers should be able to deal with this. We could however improve the functionality by adding a GUI around this with future releases.

gohabor has a much simpler way of assigning groups and we could simply say our groupnames have to be organization:team

You mean that LDAP groups need to have a colon in their name? This might be the easy way to integrate one tool like Gitea with an LDAP server. But this is not a good solution for having to integrate many services with one LDAP server.

@svenseeberg please merge main into this branch

@6543 Alright, main is merged now, as you wished

@6543 @zeripath we (mostly @melegiul ) merged the current main into the feature branch again, as there were some changes to the LDAP code in the main branch. Is there anything else we can do to get this feature ready to merge?

I and @zeripath did come to the conclusion if we would add such a feature, there should be UI support to configure for it.
So instead of a blank field there should be a configuration dialog to generate the value.
hope it was mentioned before - if not sorry for that

I have not yet come across a tool where one can configure LDAP groups with a GUI ;-) But okay, let's assume that this is really something that LDAP server operators need (personally, I disagree). Is the following approach suitable:

We stuff all available LDAP groups into an array when rendering the page (or fetch the information via API request).
First, we show a drop down with all available LDAP groups.
After selecting one LDAP group, a second drop down appears to the right with a list of existing organizations.
After selecting an org, a third drop down appears with teams within the selected org.
This can be repeated in additional "lines" (via a "plus" button) for more LDAP groups and/or different Org/Team combinations.
When all drop downs have a valid selection, we store the result as a JSON object in the background. We can use our existing infrastructure for that.

@svenseeberg how can we reach you via matrix or discord?

@svenseeberg how can we reach you via matrix or discord?

Yes :+1: , feel free to contact @sven.seeberg:matrix.netzbegruenung.de on Matrix

@zeripath Thank you for your kind review, I'm sorry that it took me so long to implement the requested changes.
I think I have covered now most of the changes, but unfortunately some are not addressed yet, concerning:

One slight fly in the ointment here is that the login_source will be read and loaded from the source quite a lot - it's infact in the DB as a JSON where it gets unmarshaled on load.

Do you agree to move the group map inside a json file in the custom folder next to app.ini?
This way we could load the map only on demand i.e. on login and on external user sync
and do not have to burden other calls to login_source, when the map is not needed

I'm really happy this feature is coming, since it's the last issue keeping us from migrating to Gitea!
Seeing it so close to merging after being open for so long is really nerve recking :-D
Is there anything I can do to help getting this in?

Just a headsup from a neighbor: replace this file with a new key to fix tests on ci: https://github.com/go-gitea/gitea/blob/main/integrations/private-testing.key

Just a headsup from a neighbor: replace this file with a new key to fix tests on ci: https://github.com/go-gitea/gitea/blob/main/integrations/private-testing.key
It expired today, breaking all Signed tests

Could you send a separate PR to fix the tests?

What's the status here? @Caellion would it be possible for you to create a PR to fix the tests? Thanks!

What's the status here? @Caellion would it be possible for you to create a PR to fix the tests? Thanks!

Seems it was already fixed in upstream about 6 hours since my comment (I was away from my proper environment at that time but someone else made such a PR)

If @melegiul or I can support you in any way, please ping us.

Looks like the tests failed due to a merge-request. Maybe this needs a rebase?

@gtudan Thanks for your support, I've merged now the latest main into this branch.

@wxiaoguang I think the remarks from your review have been addressed and the tests are green now - could you please take another look?

@wxiaoguang I think the remarks from your review have been addressed and the tests are green now - could you please take another look?

It looks good to me.
@zeripath has done much for the LDAP module, I hope he could advise more.
And since we have frontend discussions about jQuery and Fomantic-UI, I think it's better to have suggestions from @silverwind about the newly written frontend code.

@svenseeberg please do:
git merge --no-ff upstream/main # merge main branch into

and commit result of make fmt

@6543 / @svenseeberg Should we add @silverwind to the list of reviewers then?

well silverwind got a ping ... it's up to him

Sorry if I'm being pushy, but this PR is very important to my team. We are currently running our own build with this PR cherry-picked,  but that's not something we want to do in the long run.
I'd really like to help getting it in, but without the missing review for the frontend code there is not much we can do. The frontend-code seems to follow the same principals as the other code in that file - will we have to do some major refactoring for the other code in admin-common.js to remove jquery?
BTW: this PR works really well and does exactly what we need, thanks again for contributing it!

I proposed a PR  https://github.com/netzbegruenung/gitea/pull/4  (from https://github.com/wxiaoguang/gitea/commit/6ef197e4bb7c1d9f9a1755ed84fd8ce7b10da3ba)
Reason:

the LDAP group support of admin CLI is incomplete, it doesn't support group options so we do not need add our new options there.
remove TeamGroupMapEnabled, now GroupsEnabled controls both GroupFilter and our new GroupTeamMap
the UI is rewritten completely, see the screenshot, it's more simple and consistent.

@svenseeberg can you cherry-pick https://github.com/netzbegruenung/gitea/pull/4/commits/6ef197e4bb7c1d9f9a1755ed84fd8ce7b10da3ba ? I thinkt that's the cleanset solution ... & merge upstream in first

@svenseeberg can you cherry-pick netzbegruenung@6ef197e ? I thinkt that's the cleanset solution ... & merge upstream in first

Oh sorry, I read this comment after I merged the PR.
I added @wxiaoguang and @6543 as collaborators to our repo.

Since the main branch has changed a lot (including the lint rules), there are some new work to do. Give me some more time.

Hmm... CI fails, maybe I did something wrong during the refactoring. @svenseeberg could you help to take a look?

ok two things need to be done:

test if this pull works in current state for you @svenseeberg @gtudan
update inteagration tests

@svenseeberg thanks - that will help :)

🚀

Thank you all for your work on this!

Is there a way to debug this feature? It doesn't create any Organizations for me with the latest Docker image.

Is there a way to debug this feature? It doesn't create any Organizations for me with the latest Docker image.
Actually this feature is not included, orgs and teams have to be created manually
But should be not much effort to automatically generate them

Has anybody written a short manual/tutorial on how to use this feature?

Has anybody written a short manual/tutorial on how to use this feature?

This file contains an example for the Team Group Map field with the expected JSON format
https://github.com/go-gitea/gitea/blob/main/services/auth/source/ldap/README.md
Please note that you have to properly set the fields "Group Attribute Containing List Of Users" and "User Attribute Listed In Group" to detect the users groups

To elaborate a little on the previous answer: You need to have created the organizations and teams before you can sync LDAP users into them. To create the config and JSON you need to make sure that you have the following information:

the org and team names
the DN (distinguished name) of your group, for example cn=mygroup,dc=groups,dc=example,dc=com
the attribute that contains the membership information within your LDAP group object (usually something like member)
You need to know which user attribute is contained in the group membership attribute (for example the "username" cn or the full dn)

Hi @svenseeberg !
First of all, thank you for your awesome work on this! 👏
Is it possible somehow to dynamically map LDAP nested groups as organizations and teams?
Eg:
LDAP:                   | Gitea group:
- group1                | Organization 1
  - subgroup1           | Team 1
    - member 1          | Gitea user 1

And later if I create a new main group, and a subgroup (in both LDAP and Gitea) and a new member in LDAP attach that user to the team in Gitea?

I don't think that this can currently be done. The options for configuring the LDAP queries are very limited.

Hi @svenseeberg !
First of all, thank you for your awesome work on this! 👏
Is it possible somehow to dynamically map LDAP nested groups as organizations and teams?
Eg:
LDAP:                   | Gitea group:
- group1                | Organization 1
  - subgroup1           | Team 1
    - member 1          | Gitea user 1

And later if I create a new main group, and a subgroup (in both LDAP and Gitea) and a new member in LDAP attach that user to the team in Gitea?

If anyone's interested, I implemented an external solution to do this:
https://github.com/janosmiko/gitea-group-sync


Underscore character added automatically to the URL results in 404

The issue concerns this file. It contains a live link:
[About page](../Getting started/about.md)
This renders as a link to https://undp-accelerator-labs.github.io/national_innovation_ecosystems_toolkit/Getting_started/about.md, with the _ character after the word "Getting". This link is broken. The navbar resolves correctly to https://undp-accelerator-labs.github.io/national_innovation_ecosystems_toolkit/Getting started/about.md.
How can I avoid GitHub pages adding the extra character? Or, should the directory be renamed getting-started?

Fixed. It turned out it was a trivial error: the link should point to about.html, not about.md.


bug: ingress-controller crash when a HTTPRoute is created

Current Behavior
ingress-controller crash when I create a HTTPRoute
Expected Behavior
No response
Error Logs
W1107 16:25:24.909294       1 warnings.go:70] unknown field "status.conditions[0].lastTransitionTime"
2023-11-07T16:25:24+08:00	error	apisix/upstream.go:142	failed to reflect upstreamService create to cache: wrong upstream name , must contains namespace_name
W1107 16:25:24.930468       1 warnings.go:70] unknown field "status.conditions[0].lastTransitionTime"
panic: interface conversion: interface {} is nil, not *v1beta1.HTTPRoute

goroutine 557 [running]:
github.com/apache/apisix-ingress-controller/pkg/providers/gateway.(*gatewayHTTPRouteController).sync(0xc000a979e0, {0x1f424b0, 0xc0002da080}, 0xc001b22540)
	/build/controller/pkg/providers/gateway/gateway_httproute.go:157 +0x13da
github.com/apache/apisix-ingress-controller/pkg/providers/gateway.(*gatewayHTTPRouteController).runWorker(0xc000a979e0, {0x1f424b0, 0xc0002da080})
	/build/controller/pkg/providers/gateway/gateway_httproute.go:76 +0x4a
created by github.com/apache/apisix-ingress-controller/pkg/providers/gateway.(*gatewayHTTPRouteController).run
	/build/controller/pkg/providers/gateway/gateway_httproute.go:65 +0x265

Steps to Reproduce
Just apply this:
apiVersion: v1
kind: Namespace
metadata:
  labels:
    kubernetes.io/metadata.name: demo
  name: demo
spec:
  finalizers:
  - kubernetes
---
apiVersion: v1
kind: Service
metadata:
  name: httpbin
  namespace: demo
  labels:
    app: httpbin
    service: httpbin
spec:
  ports:
  - name: http
    port: 8000
    targetPort: 80
  selector:
    app: httpbin
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: httpbin
  namespace: demo
spec:
  replicas: 1
  selector:
    matchLabels:
      app: httpbin
      version: v1
  template:
    metadata:
      labels:
        app: httpbin
        version: v1
    spec:
      containers:
      - image: docker.io/kong/httpbin
        imagePullPolicy: IfNotPresent
        name: httpbin
        ports:
        - containerPort: 80
---
apiVersion: gateway.networking.k8s.io/v1beta1
kind: Gateway
metadata:
  name: httpbin-gateway
  namespace: demo
  annotations:
    cert-manager.io/cluster-issuer: selfsigned-cluster-issuer
spec:
  gatewayClassName: apisix
  listeners:
  - name: http
    protocol: HTTP
    port: 80
    allowedRoutes:
      namespaces:
        from: Same
  - protocol: HTTPS
    name: https
    port: 443
    hostname: httpbin.gigix
    tls:
      mode: Terminate
      certificateRefs:
        - name: httpbin-tls
          kind: Secret
          group: core
    allowedRoutes:
      namespaces:
        from: Same
---
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: httpbin-route
  namespace: demo
spec:
  hostnames:
  - httpbin.gigix
  parentRefs:
  - name: httpbin-gateway
    namespace: demo
  rules:
  - matches:
    - path:
        type: PathPrefix
        value: /
    backendRefs:
    - name: httpbin
      port: 8000

Environment
apisix deployed with helm chart 2.3.0
kubectl version --short
Flag --short has been deprecated, and will be removed in the future. The --short output will become the default.
Client Version: v1.25.13+rke2r1
Kustomize Version: v4.5.7
Server Version: v1.25.13+rke2r1

mark

@gigi206 I used the configuration you provided and there is no panic.

@gigi206 This bug has been fixed here https://github.com/apache/apisix-ingress-controller/pull/1825 and is available in release 1.7>=


Fix KeyValue ConfigStore set method

To resolve issue #182, The set method will now fetch any values that are contained within a set and attempt to merge them if the set is a stringified JSON and override the value if it's not. Additionally, if after the merge/override, the remaining value is an empty object or an empty string, we will attempt to delete the key to conserve resources, something that didn't work previously.

@RichardAkman Change the name of the issue and PR to something that implies what is the real fix here!


[chore] Enhance integrations docs

Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

This PR updates few images references

@leecalcote


🐛 BUG:

What version of Wrangler are you using?
2.0.25
What operating system are you using?
WSL
Describe the Bug
Wrangler publish fails with
✘ [ERROR] fetch failed

I am not sure how https://github.com/cloudflare/wrangler2/issues/834 works with latest wrangler.

Could you provide additional logging, stacktrace and context on the issue you are having? Thank you 😄

Let's say I have 1000+ files under some directories.
Total bucket size <=450MB, each file <=25MB all within KV limits.
wrangler publish either hangs or just fails with fetch failed.
There is no other error message nor logs.
I tried to do partial uploads (10 files at a time), the first few went through, but the others got stuck with the same problem as before.
Same thing happens with the web-based Drop files or even if I try to migrate to Sites (someone in Discord said Pages is no longer supported, so I tried using Sites). All of them hit this same hiccup.

Posting my comment from Discord here, as I would hate to lose it. The documentation cross-referencing along with code samples all have a myriad of Wrangler 1.0 vs 2.0 syntax, Modules vs non-Modules Workers, Pages vs Sites (so many of them don't work or just plain wrong) with many of the recommended approaches still in Beta label, so the old approach is deprecated, the new approach is just not there yet. Even the simplest getAssetsFromKV has no working example with the new module approach.

There are definitely some documentation issues that we've got on our roadmap to fix! If you could point out some specific areas we can make sure those get fixed quickly.
It sounds like for your use-case, Pages, or R2 would be a good fit (Pages is definitely not being depreciated in favour of Sites). I'll close this issue for now, but feel free to post with more details of the specific situation causing ✘ [ERROR] fetch failed and we can look into it further!

FWIW, moving to Wrangler 2, Modules+TS, R2 was painful, but well worth it at the end. The platform seems to work well, pretty smooth after the initial snafu. R2 + Workers in particular is really great, no issues at all.
Hopefully this is the last of the migrations.
Documentation is still lacking/incorrect/plain-wrong, but you are already aware of it.

Yeah, documentation in general is definitely on our roadmap to fix. If there's anything you notice that's incorrect though, do definitely point it out--that'd be really helpful!


Your chart rendered a  node which isn't supported by  yet. Please report this issue.

Describe the bug
I am trying to render AreaChart within my react-pdf report:
				<View style={styles.section}>
					<Text style={styles.sectionTitle}>Return</Text>
					<ReactPDFChart>
						<RoiChart
							investment={
								calculateEconomicParams(formData, params).investmentAfterSubsidy
							}
							annualSavings={calculateEconomicParams(formData, params).annualSavings}
						/>
					</ReactPDFChart>
				</View>

And this is my RoiChart component:
		<AreaChart
			data={chartData}
			width={600}
			height={400}
			margin={{ top: 20, right: 20, left: 20, bottom: 20 }}
		>
			<defs>
				{/* Zeleni gradijent za pozitivne vrijednosti */}
				<linearGradient id="colorPositive" x1="0" y1="0" x2="0" y2="1">
					<stop offset="0%" stopColor="#00AA00" stopOpacity={0.7} />
					<stop offset="100%" stopColor="#AAFFAA" stopOpacity={0.4} />
				</linearGradient>
			</defs>
			<CartesianGrid strokeDasharray="3 3" />
			<XAxis
				dataKey="year"
				label={{ value: "Godina", position: "insideBottom", offset: -5 }}
			/>
			<YAxis
				label={{
					value: "Povrat ulaganja (€)",
					angle: -90,
					position: "insideLeft",
				}}
			/>
			<Tooltip
				formatter={(value) =>
					`${value.toLocaleString(undefined, {
						maximumSignificantDigits: 2,
					})} €`
				}
			/>

			<Area
				type="monotone"
				dataKey="Povrat"
				stroke="none"
				width={600}
				height={400}
				fill="url(#colorPositive)"
				fillOpacity={1}
				isAnimationActive={false}
				activeDot={{ r: 6 }}
			/>
		</AreaChart>

Reproduction
You can try this type of chart to render in your component
Validations

[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that you're confident that this is a legitimate bug. To ask a question open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

Remove the <Tooltip /> component from your chart. PDFs can't have hoverable tooltips. That will fix the error.


should allow pass "data" parameter when call "tranfer" method

Steps to reproduce
When calling a contract via JSON-RPC with data, currently this gem seems not support.  after checking source code, I found it hard coded as '', shown as below:
# lib/ethereum/client.rb
  def transfer(key, address, amount)
      Eth.configure { |c| c.chain_id = net_version["result"].to_i }
      args = {
        from: key.address,
        to: address,
        value: amount,
        #  data is quite important for a Contract's manipulation, such as transfer token,
        #  please don't make it hard coded.
        data: "",
        nonce: get_nonce(key.address),
        gas_limit: gas_limit,
        gas_price: gas_price
      }

      tx = Eth::Tx.new(args)
      tx.sign key
      eth_send_raw_transaction(tx.hex)["result"]
    end

so this code snipped will not work:
require 'ethereum.rb'
require 'eth'
client = Ethereum::HttpClient.new('https://api.avax-test.network/ext/bc/C/rpc')
client.default_account = '0xa97D4D83Bb3EFE403E1e02B079A21B89947cE7A6'
client.gas_price = (25 * 1e9).to_i
private_key = ....

# let's pass the data into this transfer
client.transfer( private_key, "0xa1b2c3d4...", 0.1, data: "0xm1n2k3q4000000...")

Expected behavior
data should be passed in.
Actual behavior
data is always ""
System configuration
Ruby version:  2.6.1p33
Solc version:  --
Parity version:  --
OS:  Ubuntu 18

https://github.com/EthWorks/ethereum.rb/pull/180

@sg552
Thanks for the issue. Please migrate as we will maintain the eth.rb library here. ;)


[DRAFT] Outbox with partitioning

(wip - test is failing on purpose, expected partition = 1, actual partition = 2)

integrated into the outbox branch


[UX/UI] Citation dropdown alignment issue

Alignment issue with microsoft edge for citations on the item page, they show up aligned toward the right. Firefox aligns toward the left. Possibly related to the arrow?
Actually dropdown text needs to always be left aligned. From the comments during testing:
Lower priority issue:
Dropdown text appears to be either right or centered-aligned (?) but as a rule dropdown text should always be left aligned.
Example of issue:
https://drive.google.com/file/d/13c6mdus4jr3R0Bf3dmlf6HONG8zYIxwb/view?usp=drive_link
Also ensure the file download dropdown on the item page also follows this rule.

@wrenaria this should be ready to test on dev.


Url replacement

Hello,
I have a problem with the url option. In our project we have a variable in the template URL something like this :
bower_components/' + app.theme.name + '/app/views/activities/activity-footer.directive.html
The ngtemplate task successfully build the following structure :
$templateCache.put('bower_components/my-beautiful-theme/app/views/activities/activity-footer.directive.html'
But when i try to use the url option the structure became :
$templateCache.put('bower_components/%27%20+%20app.theme.name%20+%20%27/app/views/activities/activity-footer.directive.html'
Which, as you doubt, fail to be displayed. Do you have any idea on how to hard replace the variable in the final generated structure ?
Best regards,

Nevermind, the issue was caused by file versioning, all is great now :)


Fixed #26, returning object

Hi, thank you so much for this contribution! Would you mind signing our Contributor Licensing Agreement so we can merge the fix? You can find it here.

Done


Clarify docs on -C that it appears before the command.

The docs for -C currently don't mention that it must appear before the command name. This is the only root option that behaves this way that is documented in every page (except for +toolchain, which already mentions its position restriction). This adds some text to explain this restriction.

@bors r+

:pushpin: Commit 9ba2ca2034a20e144a3f457b214577f0fec9e27e has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 9ba2ca2034a20e144a3f457b214577f0fec9e27e with merge 34cb1ea6b8d8aaaf35b868c913cb1ff09c72a389...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 34cb1ea6b8d8aaaf35b868c913cb1ff09c72a389 to master...


How to add multiple schema.graphql files?

Hi,
I want to play around with ferry a little bit and want to add two different schema files. However I cant generate the ferry classes when I run the build_runner
File structure:

And this is my build.yaml
targets:
  $default:
    builders:
      ferry_generator|graphql_builder:
        generate_for:
          - lib/graphql/countries/**
        enabled: true
        options:
          schema: blocbloc|lib/graphql/countries/schema.graphql
          output_dir: 'graphql_generated'
          when_extensions:
            when: true
            maybeWhen: true
      ferry_generator|serializer_builder:
        enabled: true
        options:
          schema: blocbloc|lib/graphql/countries/schema.graphql
          output_dir: 'graphql_generated'
          when_extensions:
            when: true
            maybeWhen: true
  movieapi:
    builders:
      ferry_generator|graphql_builder:
        generate_for:
          - lib/graphql/movies/**
        enabled: true
        options:
          schema: blocbloc|lib/graphql/movies/schema.graphql
          output_dir: 'generated'
          when_extensions:
            when: true
            maybeWhen: true
      ferry_generator|serializer_builder:
        enabled: true
        options:
          schema: blocbloc|lib/graphql/movies/schema.graphql
          output_dir: 'generated'
          when_extensions:
            when: true
            maybeWhen: true

If I do a
flutter pub run build_runner build --delete-conflicting-outputs
I always get the following error:
Both built_value_generator:built_value and built_value_generator:built_value may output lib/client_injector.built_value.g.part. Potential outputs must be unique across all builders. See https://github.com/dart-lang/build/blob/master/docs/faq.md#why-do-builders-need-unique-outputs

The file is just a dart file that does nothing, its under /lib
What am I doing wrong?

Update:
I added
    sources:
      include:
        - lib/graphql/movies/**

to the seconds target movieapi, now the files get generated, however the serializer files seem to be not get generatred, when I add a query file

Can somebody help?

Generally, I'd recommend just connecting to one graphql API.
If you have two different servers, you could use for example Apollo Router or other means of graphql federation.
Right now, ferry only supports one schema per package. But you can create a second package just for the schema, the queries and the generated code of the second schema and import that second package from the main package

Hi @knaeckeKami
Thanks for your reply!
As it turned out, I really wont need a second schema in my current project. Would be very cool if this gets supported in future though.
This issue can be closed I guess :)


feat: integration test for openid4vp.

Signed-off-by: Volodymyr Kubiv<EMAIL_ADDRESS>

I noticed that there's no feature file like the BDD tests in the other TrustBloc projects have. I was just wondering if that was intentional (it worked better?) or if it was just an arbitrary choice


401 Client Error: Unauthorized for url: https://mainnet.infura.io/v3/...

Hi there,
I already updated dexbuytools/config/general_params.yml file with my infura project id but I keep getting:
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://mainnet.infura.io/v3/...
Do you have an idea why that is happening? Appreciate your help.

Hi,
that should mean your id is invalid, have you used it somewhere else?
also, what was the exact command you were using in the terminal?

I created a new infura id just for this project...
The command I was using is dexbuy instant eth 0x...
Folder structure:
|- dexbuytools
|-- config
|--- buy_params.yml
|--- general_params.yml
|--- wallet_data.yml

try using "pip install --editable .".
the editable-flag does not pack the source code separately and should use the right config then. I have tried it on a windows device and it worked for me (I am using windows, but I dont think it should be any different for Linux)
Since you are testing it on the eth network, a small note: the current implementation uses the flashbots network and does not send to the mainnet directly. This means that there is no guarantee that your tx goes through, but on the bright side you dont pay any gas for those tx that get not included.

That worked, thank you!
I am aware of it using flashbots and I think thats great. Does that also improve the possibility of getting filled or the main advantage is that you dont get charged for gas with tx that get not included.

I see, makes sense yea. How could I increase the possibility of getting filled in coins with just added liquidity. Those tend to spike quickly.

Increasing the bribe that is send to the Flashbots network should help. Also increasing the amount of blocks for which bundles are sent could help if your transaction did not get included in the first ones

but the flashbots network is kind of a blackbox afaik, so knowing exactly why your tx did not get in is a bit difficult

I see. So I guess changing line #111 to for i in range(1, 3) to for i in range(1, 6): and bribe to maybe 0.1?
Also I cant really know by the terminal response if the swap succeeded or not right? I'd have to check etherscan. Correct?

I see. So I guess changing line #111 to for i in range(1, 3) to for i in range(1, 6): and bribe to maybe 0.1?
Also I cant really know by the terminal response if the swap succeeded or not right? I'd have to check etherscan. Correct?

0.1 ETH bribe seems a bit over the top, I think you just need to check the average of recent successful bribes and go with something slightly above. On the flashbots website there is a statistic I iirc.
and yes, you need to check on etherscan if there is a transaction, dexbuytools will just tell you that a bundle was sent to the flashbots network

Apart from that I also tried onliquidity using the command:
dexbuy onliquidity eth --token_address 0xac4b3dacb91461209ae9d41ec517c2b9cb1b7daf
but that returns Error: No such option: --token_address.

this seems to work fine for me btw (havent performed an actual buy, but it does throw an exception at least)

Apart from that I also tried onliquidity using the command:
dexbuy onliquidity eth --token_address 0xac4b3dacb91461209ae9d41ec517c2b9cb1b7daf
but that returns Error: No such option: --token_address.

this seems to work fine for me btw (havent performed an actual buy, but it doesn't throw an exception at least)

Ok, I will test it myself again. Might've also been due to the falsely pip install from me. Will report back.

One thing I have noticed but not sure. I increased the range to 6 but I got filled twice, is that normal?

One thing I have noticed but not sure. I increased the range to 6 but I got filled twice, is that normal?

no this is definitely not supposed to happen, you sure you bought double the amount of tokens? etherscan will show you 2 txs, but one of them is just the bribe

One thing I have noticed but not sure. I increased the range to 6 but I got filled twice, is that normal?

no this is definitely not supposed to happen, you sure you bought double the amount of tokens? etherscan will show you 2 txs, but one of them is just the bribe

Yea I’m aware to see 2 tx but in my example I had 4. I might’ve ran the command too frequently since the first 1/2 did not succeed.

Would there be a way to not use flashbots? Because I keep having quite some issues with it, also given that I can never really tell what has happened in the tx. If it succeeded etc.

Would there be a way to not use flashbots? Because I keep having quite some issues with it, also given that I can never really tell what has happened in the tx. If it succeeded etc.

not in a configurable way as of now, but if you need it urgently just replace the method body of buy_instantly(...) in EthHelper.py with the same body as all other helpers:
return super().perform_uniswapv2_style_buy(token_address)


#94 Npm-xmldom-0.1.15

From #94
List of Packages with HIGH Legal risks
Packages detected
xmldom - 0.1.15
location: https://github.com/PharmaLedger-IMI/fgt-workspace/blob/master/pdm-web-components/package.json
origin: > Npm-stencil-0.0.5 > Npm-xmldom-0.1.15
Risks:
Issue Name: GPL - Copyright risk score: 6 - Patent risk score: 3 - Copyleft: Full

The requiring package was removed, and no additional visual defects where found on the future candidate release v0.10.3 (currently on https://fgt-dev.pharmaledger.pdmfc.com/ ).
Nevertheless, I ask @TiagoV-PDMFC  for a review. (See previous commit on this issue).

@TiagoV-PDMFC confirmed verbally that this library is not used. Issue closed.


Init: Reorder subsystem initialisations

Add discussed on discord, for some reason initialising the gamecontroller subsystem before the network subsystem can cause a hang in some rare cases.

Thanks!


Map Julia symbols to JavaScript strings

It is common practice to return enumeration-style results from Julia as symbols (:symbol). The parallel idiom in JavaScript is strings.
The other way around is harder: passing a JavaScript string to a Julia function should result in a Julia string, but if possible, it might be nice to also try finding a dispatch where the string is converted to a symbol if the string dispatch does not resolve.

Hmm yea, I'm not sure about the way to best solve this.  I'm leaning towards making a JavaScript type with convenient methods.  Ultimately I think it would good to have a solution that leverages the applicable features of EMCAScript 6.  To get an idea, these are partially implemented by io.js. And in the the main EMCA doc, enum will eventually be reserved.  At least something should/can be implemented quickly, but It would be nice to have a thought out path.  Maybe an intermediate solution?  I'm wary about the speed implications and 2nd guessing the Julia programmer/multiple dispatch of using jl_symbol(string) if string fails, especially if there are multiple (n) substitutions possible; there would potentially be 2^n attempts some of which might have different meaning.

I think you're right that an explicit JavaScript type that matches a Julia symbol is the clearest thing. ES6 describes such a thing, and it's supported in at least the version of node I'm using (v0.12.7). See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Symbol


Taiko Testnet

Describe the feature request
Just Lets Grow
Describe alternatives you've considered
Description of the alternatives you've considered here.
Additional context
Additional context here.

Ok


'LaneNet' object has no attribute 'Summary'

Hi, I am trying to load model LaneNet and visualize summary of layers.
code:
import keras
import math
import os.path as ops
import os
import time
import sys
import cv2
import glog as log
import matplotlib.pyplot as plt
import numpy as np
import tensorflow as tf
from config import global_config
import collections
CFG = global_config.cfg
from semantic_segmentation_zoo import cnn_basenet
from semantic_segmentation_zoo import vgg16_based_fcn
from semantic_segmentation_zoo.vgg16_based_fcn import VGG16FCN
from lanenet_model import lanenet
model = lanenet.LaneNet(phase = 'test')
model.summary()**
output:
AttributeErrorTraceback (most recent call last)
 in
21
22 model = lanenet.LaneNet(phase = 'test')
---> 23 model.summary()
24
25
AttributeError: 'LaneNet' object has no attribute 'summary'

@bcrevath The repo has beec updated. You may check the new code. Old repo was not intended to be maintained any more:)

@MaybeShewill-CV is it now possible to print the summary of layers?


Fix/broken images in notebooks

Added figures directly to repo since link embedding caused issues in notebook render

/format


add projects to upcoming

Need to add some projects to the upcoming section.

https://github.com/rooch-network

This one seems legit as well

http://bit2.network/

Suggestion here https://sequentia.io/

Brollup
Type: Sidechain Rollup
Status: Announced
Unit of Account: BTC
https://brqgoo.medium.com/introducing-brollups-18ec4081f6e7

OmniLayer
Type: CSV
Status: Launched
Unit of Account: OMNI
Explorer: https://omniexplorer.info/
Website: https://www.omnilayer.org/

Enigma Network
Type: State Channels (?)
Status: Announced
Unit of Account: BTC
Needs CTV to work
https://app.sigle.io/polydeuces.id.stx/bo-iHio5_4iTlvWwXwZ9l
https://blog.bitfinex.com/education/what-is-the-enigma-network/
https://medium.com/@IndependentAcademy/enigma-for-dummies-an-insight-into-scaling-bitcoin-like-never-before-eb83b798c7c4

thanks @stackingsaunter

Bitcoinos.build

https://www.nomic.io/

https://x.com/stargazezone/status/1806066743615574276?s=46

https://fiammachain.io/

https://www.fractalbitcoin.io/

https://www.utxostack.network/

Chains that have not been added. Need further review.
https://www.fractalbitcoin.io/
https://www.utxostack.network/
https://fiammachain.io/
https://x.com/stargazezone/status/1806066743615574276?s=46
https://www.omnilayer.org/
Enigma Network: https://blog.bitfinex.com/education/what-is-the-enigma-network/

https://yona.network/ - the solana ecosystem peeps tweeted about this one

https://auran.gitbook.io/auran-network

https://github.com/bitcoinlayers/bitcoinlayers/commit/2c44f7ec91ff7eab7a378164fdefac7a6122c965

https://www.theblock.co/post/303937/metis-co-founder-zkm-bitcoin-layer-2-goat-network

https://x.com/buildnexio

https://x.com/asulanet

https://highlayer.io/

QED

https://x.com/asulanet

bitfinity

Request from Libre community chat https://ailayer.xyz/

https://pwrlabs.gitbook.io/pwrchain/architecture-of-pwr-chain/base-layer

https://cdn.prod.website-files.com/6597cc7be68d63ec0c8ce338/664f6e01f2dd839286e20536_BitcoinX-litepaper.pdf

Chains that have not been added. Need further review.
https://www.utxostack.network/
https://fiammachain.io/
https://x.com/stargazezone/status/1806066743615574276?s=46
https://www.omnilayer.org/
Enigma Network: https://blog.bitfinex.com/education/what-is-the-enigma-network/
https://x.com/asulanet
https://auran.gitbook.io/auran-network
metis: https://www.theblock.co/post/303937/metis-co-founder-zkm-bitcoin-layer-2-goat-network
pwr/bitcoin+: https://pwrlabs.gitbook.io/pwrchain/architecture-of-pwr-chain/base-layer
https://ailayer.xyz/
BVM.network
ZKBase

Additional chains that have not been added. Need further review.
https://ligo.network/
https://eastblue.io/
infra: https://mirrorl2.com/
http://uniport.network/
https://www.bitsat.ai/
https://zkbtc.money/
https://satochain.io/
https://www.pathbtc.com/
infra: https://bool.network/
https://stratovm.io/#ecoSystem
https://ironchainbank.io/

Hemi: https://x.com/hemi_xyz

https://fehu.bio/

Looking at a few of these now. Some thoughts on things we won't add to the site:
StratoVM - launched an ERC-20 token on Ethereum. Trading is live. Running an OP Stack chain on Sepolia.
IRONChain - can't see testnet documentation anywhere.
Satochain - Not adding
Bitsat - not adding. seems ghost.
Uniport - not adding. not a lot of activity.
zkBTC - not adding right now.
path - not adding
Ligo network - not adding
Omni Layer - not adding
East blue - not adding
ZKBase - researching
Bool - researching
Fehu - researching
Marvib Labs - researching
Hemi - researching
AI Layer - researching
UTXO Stack - researching
BitcoinX - following
BVM - following
MintLayer - researching
Mirror L2 - researching
Auran - researching
Encifer - researching

Reviewing
Fehu - still not sure. continue reviewing
Bool - continue researching, but might be forced to put on site. going to play around with deposit function(s) tonight.
Marvin Labs - continue watching
AI Layer - not planning on adding. no open-source node implementation
BVM - continue watching. no updates in github on their tech
Auran - continue following
UTXO stack - continue following
MintLayer - follow roadmap and add when atomic swap protocol is live on mainnet
Mirror L2 - asking for more info
ZKBase - no bridge to bitcoin
BitcoinX - not seeing any announcement on it. will follow
Adding
Goat network - adding
Hemi - adding
encifher - adding
Spark - adding
Removed
Bit2 - seems abandoned
Highlayer - no activity
Other projects that we've reviewed, but are currently waiting for node open source before adding to site.
Satoshivm
Bouncebit

https://elastos.info/
https://syscoin.org/

closing this as we have a project for this now.


Be able to claim multiple balances at once

Thanks for providing this useful tool!
Given the inconvenience of having to copy paste and sign two transactions each time, it would be nice if multiple claimable balances could be claimed at once.
Rather than displaying a list of radio boxes, the list would need to be changed to a list of checkboxes, ideally even with a "Claim everything" option.

Thank you for your contribution.


Change DB worker type to db.r6g.xlarge

Closes #1035

When I was testing in June, the db.t4g.large was still available. Interesting...


Generate a mapping from QueryMsg to its response type

Currently, response types are generated but the mapping between QueryMsg and its responses are left to the user

I ran into some difficulty when trying to implement this. (One approach that is worth testing is generating the responses as we iterate through query messages and associate them this way.) But the core difficulty of this is

finding a reasonable way to associate the two, using any in a map/func is not ideal since the end user would still need to make the association manually. Perhaps this issue can be combined with https://github.com/srdtrk/go-codegen/issues/14.

Finding the use case for this. Maybe this is not very needed since we already generate the response types. It may be up to the user to make the association

This should be addressed with #14

Cloed with e35b998da54333320fac5aa94ec4526217ec0de3


fix(markdown): fix error of option setting

To Reviewers

[ ] Skipping review is not a problem (style, chore ONLY)
[x] Not that difficult

Type of Change

[ ] New feature
[x] Bug fixes
[ ] Feature improvement

[ ] These changes require a usage change

[ ] Others (performance improvement, refactoring, CI/CD, etc.)

Checklist

[ ] Updated Storybook documents for component
[ ] Wrote test codes for composable, utils and etc.
[ ] Tested with console(if usages are changed)

Description
SSIA
Things to Talk About

:tada: This PR is included in version 2.0.0-beta.27 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DataViewModel.showHover: false not respected

You can see this in our Toolbox example - https://toolbox-dev.xh.io/app/grids/dataview.  We don't have switches on this example to turn on/off row hover highlights, but it is off by default and we're still getting the highlight.
You can inspect the dataview grid element to verify - it has the .xh-ag-grid--no-hover class applied.
Interestingly, standard grids appear to work OK.

Note that we might want to start taking advantage of AG CSS vars....  I was able to supress this in an app with the following:
  .xh-data-view .xh-ag-grid--no-hover {
    --ag-row-hover-color: inherit;
  }

Related to #3329


rds: Add ConfigSource for CDS.

Description: Adds fields to RDS response for specifying what server to talk to for CDS.
Risk Level: Low
Testing: None
Docs Changes: Inline with API protos
Release Notes: N/A
This PR is something that seems like it would be useful, although I haven't actually discussed this with anyone yet, so I'm mainly floating this as a proposal at this point.  Please let me know what you think.
cc @htuch @mattklein123 @vishalpowar

I discussed this with @htuch and @mattklein123 yesterday, and we agreed to close this for now.  I do think that we're eventually going to need something like this anyway as part of UDPA, but we don't strictly need to deal with this right now, so we'll defer that work until later.


Fix #1322 (update facebook docs)

Thanx ;)

@jganseman Thanks! I created an issue on the (new) official documentation repository here:
https://github.com/howdyai/botkit-docs/issues/4
We'll handle it, unless someone beats us to it first!


chore: wasmCloud 0.81 bumps

chore: bump wasmcloud to 0.81
chore: bump wash-lib to 0.16
chore: bump wash-cli to 0.25

Feature or Problem
This PR bumps wasmcloud, wash-lib, and wash-cli for release. We can merge this PR but it is not quite ready to release yet as there are one or two more changes required before we release:

[ ] Bump wasmCloud component adapters in wash-lib to wasmtime rc3 from @ricochet's repo
[ ] Ensure that everything works out-of-the-box to build #1178, #1184, and our echo example
[ ] ..?

Related Issues

Release Information
wasmCloud 0.81
wash-lib 0.16
wash-cli 0.25
Consumer Impact

Testing
I was able to --dry-run publish both wash-lib and wash-cli (minus the dependency on wash-lib 0.16).
Additionally, I used the following commands to verify the proper semver release for all of these crates:
➜ convco version --bump
0.81.0
➜ convco version --prefix wash-lib-v -P crates/wash-lib --bump
0.16.0
➜ convco version --prefix wash-cli-v -P crates/wash-cli --bump
0.25.0

Unit Test(s)

Acceptance or Integration

Manual Verification

@brooksmtownsend I think the tests are failing because you bumped the version of the host, but it's not released yet

@brooksmtownsend I think the tests are failing because you bumped the version of the host, but it's not released yet

Oh yeah... that's another thing that we need to do 😅 let me break this into 2 PRs, one for the host and one for the washes

@connorsmith256 actually I ended up pinning to 0.81-rc1 so we could do our testing with it. Included in a separate commit


Pagination on releases?

It would be nice if i could specify how many releases I want to query aka  the latest release.
edit: would be to set this up for all the fields that return a list.

FYI, I'm planning on adding a pagination operator to v1.

@tyrantkhan, so for lists we can pass a per_page parameter see here: http://mikedeboer.github.io/node-github/#api-repos-getReleases.
The question is how we want to pass the parameter in the plugin options.
For me makes most sense to have somthing like:
{
 user: 'resin-io',
 repo: 'landr',
 releases: {
  limit: 1
 }
 commits: {
  limit: 100
 }
}

What do you think?


Fix support for mysql older than 5.6.5

Prior to server 5.6.5, CURRENT_TIMESTAMP cannot be used on any datatype other than TIMESTAMP

Manually merged in b63208b


Add some goto files for testing

The velociraptor check was expected to fail, since we added some base tessting files with gotos

The velociraptor check was expected to fail, since we added some base tessting files with gotos


Fix errors and missing semicolons and improve code

1. changed redundant variable and parameter names
For instance, onKeyDownHandler and onKeyUpHandler parameters have been changed to just handler. They are unnecessarily verbose because the event type (key down or key up) is already mentioned in the function name (eg., onBackspaceKeyDown)
2. fixed missing semicolons and some other, more minor errors
Probably the most important fix; the functions were missing semicolons.
3. changed easyKey (note the camelcase) to easykey
In my opinion, easykey is significantly easier to type.

There were missing semicolons, thanks for that.
However, even though I have to agree with you that just using all lowercase easykey looks better and is easier to type, there are people already using the camel case version and changing it to lowercase would cause problems for them (it would have to be a major release, e.g. from 1.0 to 2.0).
If you submit just the semicolons and renaming of the handlers I'll happily accept it.
Thanks for taking the time to do this.