VSC 1Password Debug

Summary
I've been looking to integrate the secret provision from op run -- into vscode, so I can start using it for my local development.I am using 1Password plugin for vscode that is documented here.However, there doesn't seem to be a way to integrate a tool like this (that I can think of) into vscode at the moment - specifically vscode-python.And unable to inject variables into runtime without prefix op run -- and it seems there is no option available for this.
Use cases

I Provide the secret references in .env of the format op://<vault-name>/<item-name>[/<section-name>]/<field-name>
The main the cause here is when Inject variables into runtime without the prefix of op run (using debugger), not able to inject the variables using debugger and it seems there is no option available for the debugger.

Prior Work
I have investigated a lot If a similar feature was implemented for another language, but nothing found on this issue.
References

Extension 1Password page: https://developer.1password.com/docs/vscode/
Forum of 1Password: https://1password.community/categories/cli

We have the same issue. How would one automatically resolve environment variables with op references when running a given program? Creating a resolved environment once and keeping it in the file-system seems possible but cumbersome and might easily lead to leaked secrets.

We are highly interested in having a option to use 1Password in VSCode, however the debug is butter and bread. Without a debug integration, it is not possible to use it without still saving clear text passwords.
Please add this integration or describe how to use it with the debugger :)

@MichaelMHoff @NielsKorschinsky Which debugger are you using?
Most tools allow you to configure the executable that's being run by VS Code, so you could create an op run shim:
#!/bin/sh
op run -- <whatever the original command was>

And in VS Code, point to the location of this shim.

We are using the classic launch.json (Python) of VSC.
That sounds very interesting, I tried already to achieve such thing but gave up after a few hours of googeling SOF.
Do you might have an example there?
Example launch:
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Program_Name",
            "type": "python",
            "request": "launch",
            "program": "${cwd}/main.py",
            "console": "integratedTerminal",
            "justMyCode": true,
            "args": [
                "first-arg"
                "--second_arg"
                "arg_value"
            ]
        },
]

I was able to replace the .venv/bin/python3.9 file using following scrip, made executeable:
#!/bin/sh
op run -- python3.9 $@

However the debugger is getting a timeout to connect - It is missing some kind of response.

For everyone else, who might want to spend more time on this - i found probably a solution where I currently dont have the time to try out.
https://stackoverflow.com/a/57278669

Basically it needs to support -c "import sys;print(sys.executable)" and be able to run pythonFiles\interpreterInfo.py in order to find out interpreter details that are needed.

Hey guys, many thanks for the inputs and the research. I think I could make it work, combining everything from above:
#!/bin/sh

PYTHON=$(dirname $(realpath ./python3))/python3-core

if ENV | grep "op://"; then
  # if environment contains any 1Password references...
  op run --no-masking -- "$PYTHON" "$@"
else
  "$PYTHON" "$@"
fi

It appears that the quotes around $@ are sufficient to pass everything (as-is) along to the actual python interpreter.
Use this script (made executable, chmod +x) as a replacement for python3 in your venv/bin and rename the old python3 to python3-core.
Then, configure this python3 as your interpreter in VS code and use op:// references in the env: {} part of your launch configurations.
I am using a Mac, so I figure realpath would be readlink -f for Linux-based systems.

Hi all, thanks for this awesome discussion! There's a lot of good information here, and I'm compiling all of it to put on our team's radar. Adding a debugger component to the VS Code extension is not currently on our roadmap but this is all super helpful in informing what we work on next in the extension. Thank you!

@MichaelMHoff thanks a lot for this testing! I will test that too again, however my initial test proven difficult in combination with pylance and such tools, so I really hope for a solution of the 1 password team.
@jodyheavener Thanks for getting this on your list. I really think such an native integration would be very important and helpful as this enables teams to go end-to-end from developing, testing, to deploying without having passwords in clear text.

I'm not sure if 1Password has come around on putting this in their roadmap but in the meantime as a workaround, I've found it effective to launch VS Code using op run, after which the environment variables will be available to any sub-processes launched by VS Code (including the debugger).
# Inside the project directory
op run --env-file .env -- code .

Since this largely removes the need for the .env file after VS Code is launched, it'd be nice if the argument to --env-file could be taken directly from 1Password.

@sultanofcardio this is indeed a workaround, hence not ideal. @jodyheavener is this workaround the best we can get to make 1Password secrets usable inside the VSCode debugger or do you know of any other ways combining VSCode (or any IDE?) and 1Password secrets?


New Crowdin updates

I'll get to work on this right away.

What on earth have I done...

Git is broken.

Ahhhhhh.... when you re-use a branch with the same name that GH Desktop has not realised was deleted, it will attempt to push/pull the previously deleted branch. Therefore, Desktop will automatically cache the previous branch and merge from that..  WWWWHHHHYYYYYYYY?!?!?!


Revert "Remove travel advice country page code"

Reverts alphagov/frontend#1020.
Apparently FCO is still using these pages via private-frontend.
cc @danielroseman @jennyd @boffbowsh

Here's where travel advice publisher links to these pages on private-frontend - it's all the "preview" links on the right:

FWIW, we are talking with FCO about whether these "historical previews" are actually useful; I would dearly like to remove them, so we can move to the standard preview via draft-origin.


Fix go-integration-tests in Github CI workflow.

In docker-compose.yaml, remove Armada server as dependency for executor startup, and have the former wait 30 seconds before starting, so the Executor should be running and healthy before the server expects it.  Also save docker-compose logs and Pulsar container logs to GH artifacts for debugging purposes.

Nice Job! We probably don't need all the runners to be large, but lets just get this merged.


Implement SV Concordance

I know we're not all the way there yet on deciding the business logic, so just putting this up as a placeholder.
Resolves #424
EDIT: adding reviewers now!

@bpblanken verify that Hail is definitely working on this.

Hail release is coming: https://github.com/hail-is/hail/pull/12853

Hail has released, I've updated the requirements here, and the tests now pass!
bblanken@wmbbe-67c luigi_pipeline % PYTHONPATH=.. nosetests tests/test_seqr_sv_loading.py
Initializing Hail with default parameters...
23/04/13 15:08:53 WARN Utils: Your hostname, wmbbe-67c resolves to a loopback address: <IP_ADDRESS>; using <IP_ADDRESS> instead (on interface en0)
23/04/13 15:08:53 WARN Utils: Set SPARK_LOCAL_IP if you need to bind to another address
23/04/13 15:08:54 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
Running on Apache Spark version 3.3.2
SparkUI available at http://<IP_ADDRESS>:4040
Welcome to
     __  __     <>__
    / /_/ /__  __/ /
   / __  / _ `/ / /
  /_/ /_/\_,_/_/_/   version 0.2.113-cf32652c5077
LOGGING: writing to /Users/bblanken/Desktop/code/seqr-loading-pipelines/luigi_pipeline/hail-20230413-1508-0.2.113-cf32652c5077.log
2023-04-13 15:09:03.632 Hail: INFO: scanning VCF for sortedness...
2023-04-13 15:09:31.280 Hail: INFO: Coerced sorted VCF - no additional import work to do
2023-04-13 15:09:42.969 Hail: INFO: Coerced sorted dataset     (436 + 12) / 500]
2023-04-13 15:09:51.712 Hail: INFO: wrote matrix table with 11 rows and 5 columns in 11 partitions to /var/folders/r2/68lhg74s3m130ynw3bn9wcyw0000gq/T/tmpueyf8q19/tmp8x4fl6gf.mt
    Total size: 2.93 KiB
    * Rows/entries: 2.79 KiB
    * Columns: 42.00 B
    * Globals: 107.00 B
    * Smallest partition: 1 rows (199.00 B)
    * Largest partition:  1 rows (318.00 B)
2023-04-13 15:09:55.787 Hail: INFO: wrote matrix table with 11 rows and 5 columns in 11 partitions to /var/folders/r2/68lhg74s3m130ynw3bn9wcyw0000gq/T/tmpueyf8q19/tmp_ostpi8k.mt
    Total size: 1.93 KiB
    * Rows/entries: 1.78 KiB
    * Columns: 42.00 B
    * Globals: 107.00 B
    * Smallest partition: 1 rows (152.00 B)
    * Largest partition:  1 rows (215.00 B)
.Stage 11:>                                                       (0 + 11) / 11]
----------------------------------------------------------------------
Ran 1 test in 71.787s

OK


Yet another bbob

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue
We need something close to the various benchmarks with name close to "bbob".
How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CONTRIBUTING).
[x] All tests passed, and additional code has been covered with new tests.

Sorry reviewers, I merge now, I might need it with some users today...


Fix JS API client

I realized that in #90, I left the javascript-sdk in a state where it doesn't compile. My reasoning for not including javascript-sdk/src/api.ts in that PR was because the server API had not been updated yet, so the interface would be wrong if I included these changes in api.ts. I forgot to take into account that the types wouldn't match and would cause a compile error. I'm including those changes now because I think it's better to have the API client not match than to have the whole project not compile. Really I just need to stop submitting breaking changes :grimacing:
I also added npm run build to the CI checks to prevent this sort of thing in the future.

@seanpmorgan friendly bump


Document highlighting doesn't work in notebooks

Open the notebook in the pylance-release/testing/single folder
Scroll down to the highlight test and stick the cursor in the __init__ function:

It should be highlighting like the source file does:

Could be the same issue as https://github.com/microsoft/pylance-release/issues/5524

Also in a notebook it seems to be doing string matching:

As opposed to what happens in a .py file:


[vcpkg] Fix build with feature deps

Hi @ras0219-msft !
It appears that magnum is the first package that depends on features of other packages. (e.g. magnum on corrade[utility]). Up till now, these dependencies are not parsed as features, but packages, and a package called "corrade[utility]" is obviously never installed.
This PR fixes that and allows the build command to find the installed features of a package when listed in Build-Depends.
Cheers,
Jonathan.

Hi @Squareys ,
We have the same problem with PCL where the qt feature depends on vtk[qt]. These issues seem related to this problem #2430 #2743 #2440 .
I will give your PR a try to see if it solves my problem

@maxula Be aware though, that this only fixes the build command! Not install. I'm having problems with that aswell (debugging through it currently).

LGTM!
I've also had trouble reproducing the transitive feature dependencies bug -- they appear to work in all my tests. @Squareys do you have a repro you could share?

@ras0219-msft Thanks for merging!
I am unsure which issue you are referring to by "". Is it https://github.com/Microsoft/vcpkg/issues/2430?
My issue with install was solved with https://github.com/Microsoft/vcpkg/pull/2819 :)


Specify that we provide TEXT instead of MD or HTML

As the lines are joined by simple line-breaks, we should let Apprise know that it is text. If one of the outgoing providers wants HTML, Apprise will then care of correct formatting, as was the case in issue #347

Closing out all these old PRs. Feel Free to re-open if still valid.


Can a new release be published

While trying out delta-sharing:1.0.0 I ran into some bugs which are resolved by a few merged PR's in main.
The most important (to me) are:

Able to add hadoop core-site.xml config to the delta sharing server #42
Fix load_as_pandas on table without stats #30
Use docker image from sbt docker:publishLocal

would be lovely to have a new release (1.1.0 or 2.0.0??) ( for the python/scala packages at least. A docker image on docker hub would be even more awesome

Yep. We will do a new release 0.2.0.

Closing this as 0.2.0 is out.


VMR unified build fails on PGO legs: Download from all targets failed

https://dev.azure.com/dnceng/internal/_build/results?buildId=2448099&view=results
    /vmr/src/sdk/src/Installer/redist-installer/targets/GenerateLayout.targets(400,5): error : Download from all targets failed. List of attempted targets: file:///vmr/artifacts/assets/Release/Runtime/9.0.0-preview.5.24256.1/dotnet-runtime-9.0.0-preview.5.24256.1-linux-x64.tar.gz [/vmr/src/sdk/src/Installer/redist-installer/redist-installer.csproj]
    /vmr/src/sdk/src/Installer/redist-installer/targets/GenerateLayout.targets(400,5): error : Failed to download file using addresses in Uri and/or Uris. [/vmr/src/sdk/src/Installer/redist-installer/redist-installer.csproj]

Happens on both Windows and Linux.
/cc @ViktorHofer @jkoritzinsky

@jkoritzinsky I think it was you who added the PGO legs? Please take a look.


[Feature/Bug] Add option to switch among vertical and horizontal split for new window

Is your feature request related to a problem? Please describe.
I like the binary tree and half options to automatically add a new window to the right, however I find it limited compare to other window managers (I understand that this is a script and not a proper window manager).
Describe the solution you'd like
I would like to have the option to "togglesplit" between horizontal and vertical, meaning that if I have two windows side by side with togglesplit the windows would move to be up and down. I find this nice in cases where I have two terminals open (on at the top and on at the bottom, however I like the default side by side configuration for everything else).

+1 to this. exact same use case :D


Ability to run user ROS code on internal A1 computer

Hi,
Is it possible to run our ROS nodes on the internal computer inside of the A1 or do we require an external computer? For instance, if I wanted to run the ROS 2 navigation stack with a waypoint follower node, would that have to be done on a separate machine connected over ethernet and share the ROS network or could I add it to the robot computer? How much compute power is left on the internal computer after the system operations?

Hi, does your A1 has a NX? If so you can try to run your ROS2 node on NX. If your A1 just have a Raspberry Pi, we afraid that the power is not enough.


[Example] ggml: add command-r example

Add a Command-R example along with the corresponding tests.

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


ARROW-7616: [Java] Support comparing value ranges for dense union vector

After we support dense union vectors, we should support range value comparisons for them.

@TheNeuralBit do you have time to review?

LGTM. Thanks for the contribution and for your patience :)

Thanks a lot for your effort.


Rework node contract deployment hash/data

Deployment hash should be fixed size 32 bytes (h256)
Deployment data should have a max size of 512 bytes (to be changed in later version)

Deployment hash
For the contract hash the default value in polkadot is 64 bytes not 32
and i was able to create multiple contracts with this format from extrinsics -> smartContractModule -> createNodeContract, contracts (6784,6785,6786)

and when i checked the deployment hash of an already existing contract created through the grid it was 32 not 64

Deployment Data
for the deployment data the max size was verified as 512 i tried creating a contract with 512 characters and it was created successfully and then i tried to create one with 513 characters and the creation failed.

This should be reworked to accept 16 bytes instead of 32. The clients make an md5 hash of the deployment and then hexify it, essentially doubling the data from 16 to 32.
The default value: 0x000.. is not 64 bytes by the way, it's just the zero representation of a 32 byte array

Verified on devnet.
i used a 32 bytes format like the one used on other contracts created on the grid and it worked.

and it only accepted 32 bytes. and when i tried to enter 31 or 33 it was not accepted.


#1401 Down arrow event in korean input mode.

Description
When selecting korean column name in the suggestion layer , Down Arrow goes to the first column.
한글 컬럼 이름 suggetion을 선택할때, 아래 화살표를 입력하면 첫번째 컬럼으로 간도록 수정
Related Issue :
#1401
How Has This Been Tested?

menu > MANAGEMENT > DataPreparation.
click Dataflow > click Wrangled Data > Edit rules.
select rule  aggregate, pivot and window.
add function and then add korean column name.
select column name in suggestion layer.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project. it will be added soon
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document. it will be added soon
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Additional Context

run build
deploy to 4


[WIP] Kafka channel fix scope

Fixes #1189

Proposed Changes

Release Note

Docs

I wanted to create this PR for fixing https://github.com/knative/eventing-contrib/issues/1189 but as per the comment https://github.com/knative/eventing-contrib/issues/1189#issuecomment-624281888, I will close.


[Feature Request] Add manifest.json into versionFile

I want to update manifest.json version in chrome extension project.
It is probably the same as package.json.
Thanks.

Thank you for the issue.
Supported at #126 and released as v1.0.4.
However, if your project already has a .tagpr configuration file in place, no additional version file detection proceeds, so set the version file by yourself as follows
git config -f .tagpr tagpr.versionFile manifest.json

Also note that the version number set in the file must be in semver format, such as 1.2.3.
Please try it.

@Songmu
Thank you for the quick response!
I tried it on my project and it worked fine.

I see. You wanted to update multiple version files. I appreciate you making good use of it.


Labels as int64 (on Mac) aren't able to be saved with AICSImage OmeTiffWriter

Detected when downloading 0.0.4 on a colleagues Mac. The default on windows is int32 for labels layers. This image type is independent of how the underlying image layers are loaded or the original file type.

Currently using AICSImage.save works to save int32. However, automated dim selection is not ideal and would rather use AICSImage.writers.OmeTiffWriter, but work needs to be done on organizing the meta data

Updated dim selection in v0.0.5. Closing


Anki 2.1.49

Hi
So, I am using this frequency dictionary JPDB and this is the anki before using the add-on:

This is after using it:

You can see the first card due changed for this number #100000
Only one card changed, but usually all cards change to this number
I reset the cards clicking in ''forget'' and tried again and this time nothing changed

I see, the addon is marking that card as not being in the dictionary. Can you send me a screenshot of how your preferences are set? Also if you could export this deck and send the file to me it would be much easier to test, you can drop it in this same thread.

My preferences are set like this:

This is the deck I used to do the tests:
https://mega.nz/file/KAwTURSL#9kqlOzbiMZueBOM53N-bcLfJI1HaSPnNOv3SyhzoaWI

Can you check the "Modify" box in your preferences and try reordering again?

I checked the box and tried again:

The first photo is before reorder and the second after reorder
I also tried again changing ''sorting mode'' field rank to occurrence, but nothing changed

There was an issue with cards having hidden things in the field, so for example the card that had "化合物" in the Word field actually had "化合物", which made the dictionary not be able to match the word.
I pushed a new update that fixes this, you can try it out by updating the addon, going to your preferences and selecting the option to ignore HTML tags. You'll also want to check the box with the option to schedule a refresh so the cards are updated.
Please try the fix and let me know if you encounter any problems.

I really don't know why this ''< div >'' appears in the field ''Word''
I tested it again and now it seems to be working fine
You did a good job, thank you very much


does anyone try to deploy this great repo in jetson orin? Unsupported operator Gridsampler2DTRT

first of all, thanks for this great job, i 've deployed this project in x86 ubuntu system successfully,  however, i  encountered lots of problems when i tried to deploy it in jetson orin pack.  for example, it showed Unsupported operator Gridsampler2DTRT during running test_trt_ops.py even i installed onnx 1.12.0, torch 1.12.0 in orin jetson pack.     i noticed that someone has encountered similar problem as i showed, it was solved via installed onnx1.12 and torch 1.12 synchronously. but the difference is that i installed tensorrt<IP_ADDRESS> in orin!!
i don't know whether the difference of tensorrt version would result in such issue,  by the way, i also encountered nan_to_num plugin issue as well. thanks alot, if you can help me

It looks like the custom plugin dynamic link library is not loaded properly. Are you sure loaded BEVFormer_tensorrt/TensorRT/build/libtensorrt_ops.so?

@DerryHub thanks for your reply， i am sure that i 've already loaded *ops.so successfully judged via logger's printing, by the way, i have already addressed such issue, i substitude this nan_to_num function by using np.nan_to_num for original function in that of torch, and then it converted onnx into trt successfully, thanks a lot any way

nan_to_num
Hello, which side did you modify to solve the problem?
I try to modify the encoder of det2trt, and the trt can be converted successfully, but the indicator result is incorrect.

@Alex-fishred  it is really a long time issue i encountered, i don't remember where i modified in code exactly,     but i have two options for you,  1, write the declaration of nan_to_num plugin in hpp and  definition  of nan_to_num plugin in CPP, and register it in "So" appropriately.  2,   option2,  substitute it with numpy version of nan_to_num instead of torch version of "nan_to_num".    But it will result in the  decrease of inference accuracy, you have to figure out a way to solve it.

Thank you for your reply, thank you very much. I have tried np's nan_to_num, and it can indeed be successfully inferred but the accuracy chance is 0. Do you have any suggestions on how to correct the accuracy?  I probably won’t implement option 1 because I don’t know C++ very well.

Thank you for your reply, thank you very much. I have tried np's nan_to_num, and it can indeed be successfully inferred but the accuracy chance is 0. Do you have any suggestions on how to correct the accuracy? I probably won’t implement option 1 because I don’t know C++ very well.

i also have no idea about how to increase the accuracy, sorry about that, it seems that it is probably helpful to do post-nms, or filter results by tuning threshold of confidence....  Do you have any progress about this issue?

Hello, @sainttelant what do you mean by installing onnx1.12 and torch 1.12 synchronously? I also have the same problem unsupported operators even when I have built custom plugins, mmdeploy.

@vietnguyen012 , you have to install the corresponding versions onnx1.12 and torch1.12 in your Jetson devices, otherwise you will encounter many incompatible ops.

@sainttelant have you converted successfully custom plugins?

@vietnguyen012 actually, i 've written my own custom plugins, and i am sure i 've registered the plugins there, of course, it converted successfully, however, it still couldn't find my plugins when i executed the instances of the functions, it is weird, finally,  i replaced an ops called num_to_nan as numply function, it can execute the inference at last, however the accuracy  decreased too much.  i will show you my docker image here, you can refer to it. https://hub.docker.com/repository/docker/sainttelant/bevformer_xw_tensorrt/general


don't support css3 @keyframes

@keyframe still choking less.  @-moz-keyframes and @-webkit-keyframes work though.
Don't think this is a massive issue at the moment as nothing supports @keyframe without the vendor prefix but would be nice to have so I can add the code in and forget stop worrying about it.


Added reset password flow UI

Fixes #546
Signed-off-by: Abhishek P<EMAIL_ADDRESS>

Closing in favor of #636.


Fix bug when running watsonx_llm models

When invoking a watsonx_llm eval directly, a TypeError occurs at this line where the function is expecting a str, but a dict is passed instead.
This PR fixes that error by assuming the arguments to create_from_arg_string match those of the parent method, but allowing for the existence of a string as well. I'm not sure what the scenario looks like in which this value is a string, which is why I would appreciate review from @Medokins to confirm as I know that the PR #2397 delivering support for watsonx works for his purposes.

Closing in favor of #2464


Could not load UnityEngine.Texture2D

Any time I craft apparel I get this red error. I don't know if anyone else has reported this.
https://gist.github.com/AGingersaurusRex/200aebef563f3db8fb36f9b7f2c48b89

Nice job finding this, if it is indeed reproducible. I've been trying to hunt this one for a while.

Closing bug. Many bugs have been fixed since this was reported. Please reopen in new bug report using the new templates if this issue persists.


Rebuild sidebar order

Sidebar items order were wrong:

They look fine after this fix:

I created this branch from rebuild, not sure about those 3 first commits :/

This doesn't work.


Family id filter fix

Small UI fixes

Codecov Report
Patch and project coverage have no change.

Comparison is base (eb34453) 65.32% compared to head (5ddf04f) 65.32%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #439   +/-   ##
=======================================
  Coverage   65.32%   65.32%
=======================================
  Files          73       73
  Lines        6261     6261
=======================================
  Hits         4090     4090
  Misses       2171     2171

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


tunnel routing "connection refused"

The problem
Recently set up the addon with o problems for the local system, but when I'm adding additional hosts I will get and error.
What version of Cloudflared has the issue?
4.1.1
What was the last working version of Cloudflared?
4.1.1
What type of installation are you running?
Home Assistant OS
Add-on YAML Configuration
external_hostname: my.domain.com
additional_hosts:
  - hostname: myother.domain.com
    service: http://<IP_ADDRESS>:80
tunnel_name: homeassistant

Anything in the logs that might be useful for us?
-----------------------------------------------------------
 Add-on: Cloudflared
 Use a Cloudflare Tunnel to remotely connect to Home Assistant without opening any ports
-----------------------------------------------------------
 Add-on version: 4.1.1
 You are running the latest version of this add-on.
 System: Home Assistant OS 9.5  (amd64 / generic-x86-64)
 Home Assistant Core: 2023.3.5
 Home Assistant Supervisor: 2023.04.0
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums or the Discord chat.
-----------------------------------------------------------
[13:24:51] INFO: Checking add-on config...
[13:24:51] INFO: Checking for existing certificate...
[13:24:51] INFO: Existing certificate found
[13:24:51] INFO: Checking for existing tunnel...
[13:24:51] INFO: Existing tunnel with ID (some numbers) found
[13:24:51] INFO: Checking if existing tunnel matches name given in config
[13:24:52] INFO: Existing Cloudflare Tunnel name matches config, proceeding with existing tunnel file
[13:24:52] INFO: Creating config file...
[13:24:52] INFO: Validating config file...
Validating rules from /tmp/config.json
OK
[13:24:52] INFO: Creating DNS entry my.domain.com...
2023-04-16T20:24:53Z INF my.domain.com is already configured to route to your tunnel tunnelID=(some numbers)
[13:24:53] INFO: Creating DNS entry myother.domain.com...
2023-04-16T20:24:53Z INF myother.domain.com is already configured to route to your tunnel tunnelID=(some numbers)
[13:24:53] INFO: Finished setting up the Cloudflare Tunnel
[13:24:53] INFO: Connecting Cloudflare Tunnel...
2023-04-16T20:24:53Z INF Starting tunnel tunnelID=(some numbers)
2023-04-16T20:24:53Z INF Version 2023.4.0
2023-04-16T20:24:53Z INF GOOS: linux, GOVersion: go1.19.6, GoArch: amd64
2023-04-16T20:24:53Z INF Settings: map[config:/tmp/config.json cred-file:/data/tunnel.json credentials-file:/data/tunnel.json loglevel:info metrics:<IP_ADDRESS>:36500 no-autoupdate:true origincert:/data/cert.pem]
2023-04-16T20:24:53Z INF Generated Connector ID: "some numbers"
2023-04-16T20:24:53Z INF Initial protocol quic
2023-04-16T20:24:53Z INF ICMP proxy will use <IP_ADDRESS> as source for IPv4
2023-04-16T20:24:53Z INF ICMP proxy will use :: as source for IPv6
2023-04-16T20:24:53Z INF Starting metrics server on [::]:36500/metrics
2023-04-16T20:24:54Z INF Connection (some numbers) registered with protocol: quic connIndex=0 ip=<IP_ADDRESS> location=PHX
2023-04-16T20:24:54Z INF Connection (some numbers) registered with protocol: quic connIndex=1 ip=<IP_ADDRESS> location=LAX
2023-04-16T20:24:55Z INF Connection (some numbers) registered with protocol: quic connIndex=2 ip=<IP_ADDRESS> location=LAX
2023-04-16T20:24:56Z INF Connection (some numbers) registered with protocol: quic connIndex=3 ip=<IP_ADDRESS> location=PHX
2023-04-16T20:28:06Z ERR  error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" cfRay=7b8f2c3be88f7c1e-LAX ingressRule=1 originService=http://<IP_ADDRESS>:3012
2023-04-16T20:28:06Z ERR Request failed error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" connIndex=1 dest=https://myother.domain.com/ ip=<IP_ADDRESS> type=http
2023-04-16T20:28:06Z ERR  error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" cfRay=7b8f2c3c99007c1e-LAX ingressRule=1 originService=http://<IP_ADDRESS>:3012

Steps to reproduce the issue
I set up the addon by adding into configuration the external home assistant host and Cloudflair tunnel name. this worked and I have no problems with connecting to home assistant. I wanted to access other services on the network as well so I added into additional hostnames the information as shown in the YAML. this is when I get the error. I can still access Home assistant with no problems.
Additional information
No response

This is indeed strange.
What I notice at first glance is that you have configured the service http://<IP_ADDRESS>:80 in the additional_hosts. The error message from the log however tries to connect to <IP_ADDRESS>:3012.
Does more helpful information appear in the log when you change the log level in the add-on options to debug?

Additionally, can you add a little bit more context about what kind of service you are trying to route through the tunnel?
Some services do block requests from unknown reverse proxies (e.g. new routers like Fritz!Box), so this might be an issue on the service side.

This is indeed strange. What I notice at first glance is that you have configured the service http://<IP_ADDRESS>:80 in the additional_hosts. The error message from the log however tries to connect to <IP_ADDRESS>:3012. Does more helpful information appear in the log when you change the log level in the add-on options to debug?

sorry that was me trying something while I was typing out the message. it should be port :3012

it's a docker on an Unraid server. The service is vaultwarden:  https://github.com/dani-garcia/vaultwarden/wiki

So I guess 3012 is the websocket port of Vaultwarden, right?
Can you try to use ws://<IP_ADDRESS>:3012 for your service definition?

ws://<IP_ADDRESS>:3012

under the other hostname section?

hostname: gardaz.blitzthenet.homes
service: ws://<IP_ADDRESS>:3012

Exactly... But it is just a guess on my part, after a short Google research.
https://community.cloudflare.com/t/tunnel-with-websockets/318111
https://github.com/cloudflare/cloudflared/issues/526

this is what i'm gettign
2023-04-16T21:38:08Z ERR  error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:8123: connect: connection refused" cfRay=7b8f92d34cf77eb7-LAX ingressRule=0 originService= (XX)
2023-04-16T21:38:08Z ERR Request failed error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:8123: connect: connection refused" connIndex=2 dest=https:// (XX) milleraz.blitzthenet.homes/api/websocket ip=<IP_ADDRESS> type=ws

The error message refers to the 8123 port, where Home Assistant is normally active. Also the IP address 172.30.xx is the Home Assistant OS internal IP of the Home Assistant Docker container.
Can you provide more information on what the add-on configuration looks like now?
And a question out of interest, is there a reason why you install Vaultwarden separately in Docker and not use the add-on in Home Assistant?

Are you referring to vaultwarden, cloud flair, or somthing else in regards to add-on configuration.

The last logs you provided are independent of Vaultwarden because they contain the address and port of Home Assistant.
Could you change your add-on configuration to something like this and try again?
external_hostname: my.domain.com
additional_hosts:
  - hostname: myother.domain.com
    service: ws://<IP_ADDRESS>:3012
tunnel_name: homeassistant

What I'm getting now.
[15:10:50] INFO: Creating DNS entry my.domain.com...
2023-04-16T22:10:51Z INF my.domain.com is already configured to route to your tunnel tunnelID= ( Some number)
[15:10:51] INFO: Creating DNS entry myother.domain.com...
2023-04-16T22:10:51Z INF myother.domain.com is already configured to route to your tunnel tunnelID=5( Some number)
[15:10:51] INFO: Finished setting up the Cloudflare Tunnel
[15:10:51] INFO: Connecting Cloudflare Tunnel...
2023-04-16T22:10:51Z INF Starting tunnel tunnelID= (some number)
2023-04-16T22:10:51Z INF Version 2023.4.0
2023-04-16T22:10:51Z INF GOOS: linux, GOVersion: go1.19.6, GoArch: amd64
2023-04-16T22:10:51Z INF Settings: map[config:/tmp/config.json cred-file:/data/tunnel.json credentials-file:/data/tunnel.json loglevel:info metrics:<IP_ADDRESS>:36500 no-autoupdate:true origincert:/data/cert.pem]
2023-04-16T22:10:51Z INF Generated Connector ID:  (some number)
2023-04-16T22:10:51Z INF Initial protocol quic
2023-04-16T22:10:51Z INF ICMP proxy will use <IP_ADDRESS> as source for IPv4
2023-04-16T22:10:51Z INF ICMP proxy will use :: as source for IPv6
2023-04-16T22:10:51Z INF Starting metrics server on [::]:36500/metrics
2023-04-16T22:10:52Z INF Connection  (some number) registered with protocol: quic connIndex=0 ip=<IP_ADDRESS> location=PHX
2023-04-16T22:10:52Z INF Connection (some number) egistered with protocol: quic connIndex=1 ip=<IP_ADDRESS> location=LAX
2023-04-16T22:10:53Z INF Connection (some number) registered with protocol: quic connIndex=2 ip=<IP_ADDRESS> location=LAX
2023-04-16T22:10:54Z INF Connection (some number) registered with protocol: quic connIndex=3 ip=<IP_ADDRESS> location=PHX
2023-04-16T22:15:04Z ERR  error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" cfRay=7b8fc8ea0ec77d5f-LAX ingressRule=1 originService=ws://<IP_ADDRESS>:3012
2023-04-16T22:15:04Z ERR Request failed error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" connIndex=1 dest=https://myother.domain.com/ ip=<IP_ADDRESS> type=http
2023-04-16T22:15:04Z ERR  error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" cfRay=7b8fc8ed89917d5f-LAX ingressRule=1 originService=ws://<IP_ADDRESS>:3012
2023-04-16T22:15:04Z ERR Request failed error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" connIndex=1 dest=https://myother.domain.com/ ip=<IP_ADDRESS> type=http
2023-04-16T22:15:04Z ERR  error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" cfRay=7b8fc8eeaa5d7d5f-LAX ingressRule=1 originService=ws://<IP_ADDRESS>:3012
2023-04-16T22:15:04Z ERR Request failed error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp <IP_ADDRESS>:3012: connect: connection refused" connIndex=1 dest=https://myother.domain.com/favicon.ico ip=<IP_ADDRESS> type=http

Currently I can't think of anything else we could try, sorry.
Since it's not directly a problem with the add-on, but with Cloudflare itself, I move the error to the discussions.


Updated SPI/I2C samples to use "GetDefaultAsync" rather than do the D…

…eviceInformation dance

Hi @jessekaplan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


In the "change a license" section, suggest to also add best practice for license changes in a group-based licensing scenario

In the "change a license" section, suggest to also add best practice for license changes in a group-based licensing scenario

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

ID: 3275e03f-6c8e-a61f-2d0b-52b0bc45e0d7
Version Independent ID: 3275e03f-6c8e-a61f-2d0b-52b0bc45e0d7
Content: Phone numbers and licensing changes - Microsoft Teams
Content Source: Teams/phone-numbers-licensing-changes.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

When can we expect the expansion with "best practice for license changes in a group-based licensing scenario"?

Hi @davidjfriedman, thank you for your feedback.
I'll forward your feedback, but I cannot promise that the content will be added to the article because this is a very specific situation. May I suggest you contact support or your TAM for more proactive help with this issue.

Hi @adrianfleisch Hope all is well. Please let us know if there are any additional questions.

Hi @davidjfriedman, thank you for your feedback.
I'll forward your feedback, but I cannot promise that the content will be added to the article because this is a very specific situation. May I suggest you contact support or your TAM for more proactive help with this issue.

Thanks. but it's not a issue. It's just a suggestion to enhance the documentation with additional information, because a customer asked about it.
@scanum It's ok for me. Have no additional questions.

@davidjfriedman and @adrianfleisch,
I think the important information for your topic is:

From this article: "If you need to change a license for a user that involves one of the prerequisite licenses, you should ensure that the licensing changes are made and saved at the same time. This method will ensure that the user keeps their assigned phone number and can continue making and receiving PSTN calls in Microsoft Teams."
From https://learn.microsoft.com/microsoftteams/phone-numbers-licensing-changes, "When you update license assignments for a user or group, the license assignment removals and new assignments are made simultaneously so that users do not lose access to their services during license changes or see license conflicts between plans."


Make Player Contain a sf::view

sf::view centers the world around its position. To make the world move when the player moves a sf::view should be attached to the players position.

Not really required for demo anymore.

This is more of a game specific issue


Add npm5 git support

Wanted to use my update but you haven't released it yet so would be nice to just install it directly from your git without the need to publish (something I miss from bower) :)

Hey @RobinBuschmann I think this is a better idea than the automatic release that we have been discussing.
npm docs state:
Git dependencies with prepare scripts will now have their devDependencies installed, and their prepare script executed as if under npm pack.

We would be only affecting users that install via git. I like this idea @intellix !

Codecov Report

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

@@           Coverage Diff           @@
##           master     #202   +/-   ##
=======================================
  Coverage   96.71%   96.71%
=======================================
  Files         115      115
  Lines        1004     1004
  Branches      129      129
=======================================
  Hits          971      971
  Misses         15       15
  Partials       18       18

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 46ae4ca...155ba3e. Read the comment docs.

Looks good! Let's do it!


Unable to import extensions in Python 3.12 / ModuleNotFoundError

After upgrading to Python 3.12, importing an extension, for example from bottle.ext.websocket import websocket, now gives the following error:
Traceback (most recent call last):
  File "C:\...\test.py", line 1, in <module>
    from bottle.ext.websocket import websocket
ModuleNotFoundError: No module named 'bottle.ext.websocket'

My environment:

Windows 10
Python 3.12.0
bottle 0.12.25
bottle-websocket 0.2.9 (the example extension)

Reproduction:

Install dependencies: pip install bottle==0.12.25 bottle-websocket==0.2.9
Run script:from bottle.ext.websocket import websocket

Investigation / Potential Solution:
I see this error does not occur on master but it does on the release-0.12 branch / 0.12.25 tag. Using the changes from https://github.com/bottlepy/bottle/commit/ca6762c559c5e71e0dff71dc97eb4c6b3ed9bbcd that are currently on master seems to fix this issue.

To add more info to this, I see support for the find_module API has been removed in Python 3.12 as noted in the changelog here:

Support for find_loader() and find_module() APIs have been removed. (Contributed by Barry Warsaw in gh-98040.)

The changelog recommends using find_spec() instead.

This is already implemented in 0.13-dev and will be released soon.


Error: "Value was either too large or too small for an Int32 - 'open_interest'"

Ticker entity has property OpenInterest with type int, that has too small size to deserialize data from deribit into the Ticker response.
Full error message:
Error processing message {"jsonrpc":"2.0","method":"subscription","params":{"channel":"ticker.BTC-PERPETUAL.raw","data":{"timestamp":1613222943860,"stats":{"volume_usd":193698960.0,"volume":4088.6392549,"price_change":-2.2837,"low":45250.0,"high":48624.5},"state":"open","settlement_price":47157.09,"open_interest":2446540246,"min_price":46191.37,"max_price":47598.22,"mark_price":46896.7,"last_price":46895.5,"instrument_name":"BTC-PERPETUAL","index_price":46881.37,"funding_8h":0.00052489,"estimated_delivery_price":46881.37,"current_funding":0.0,"best_bid_price":46898.0,"best_bid_amount":90.0,"best_ask_price":46898.5,"best_ask_amount":114580.0}}
Newtonsoft.Json.JsonReaderException: Could not convert to integer:<PHONE_NUMBER>. Path 'open_interest'.
---> System.OverflowException: Value was either too large or too small for an Int32.
I'd like to make PR with this fix, could you review it?

Thanks Onikiro!


Submariner support for Dual Stack networks

This PR includes the following changes.

Support in mtu_handler to skip processing IPv6 addresses.
Local privateIP resolution code now uses only IPv4 addresses while creating an endpoint.
Support in Submariner Gateway pod for dual-stack clusters
Support in Submariner Route-agent pod for dual-stack clusters

Partially Fixes: https://github.com/submariner-io/submariner/issues/1739
Signed-off-by: Sridhar Gaddam<EMAIL_ADDRESS>

Sample output of subctl show all in a dual-stack k8s cluster:
[sgaddam@localhost submariner]$ subctl show all
Cluster "cluster1"
 ✓ Detecting broker(s)
NAMESPACE               NAME                COMPONENTS     GLOBALNET   GLOBALNET CIDR   DEFAULT GLOBALNET SIZE   DEFAULT DOMAINS
submariner-k8s-broker   submariner-broker   connectivity   no          <IP_ADDRESS>/8      65536

 ✓ Showing Connections
GATEWAY           CLUSTER    REMOTE IP    NAT   CABLE DRIVER   SUBNETS                        STATUS      RTT avg.
cluster2-worker   cluster2   <IP_ADDRESS>   no    libreswan      <IP_ADDRESS>/16, <IP_ADDRESS>/16   connected   388.079µs

 ✓ Showing Endpoints
CLUSTER    ENDPOINT IP   PUBLIC IP         CABLE DRIVER   TYPE
cluster1   <IP_ADDRESS>    <IP_ADDRESS>   libreswan      local
cluster2   <IP_ADDRESS>    <IP_ADDRESS>   libreswan      remote

 ✓ Showing Gateways
NODE              HA STATUS   SUMMARY
cluster1-worker   active      All connections (1) are established

 ✓ Showing Network details
    Discovered network details via Submariner:
        Network plugin:  kindnet
        Service CIDRs:   [<IP_ADDRESS>/16,fd00:1234::/112]
        Cluster CIDRs:   [<IP_ADDRESS>/16,fd00:200::/48]

 ✓ Showing versions
COMPONENT               REPOSITORY           VERSION
submariner-gateway      localhost:5000       local
submariner-routeagent   localhost:5000       local
submariner-operator     quay.io/submariner   devel

All e2e test-cases passed:
[SynchronizedAfterSuite]
github.com/submariner-io/shipyard@v0.15.0-m2/test/e2e/e2e.go:54
[SynchronizedAfterSuite] PASSED [0.000 seconds]
------------------------------

Ran 12 of 44 Specs in 164.556 seconds
SUCCESS! -- 12 Passed | 0 Failed | 0 Pending | 32 Skipped
[sgaddam@localhost submariner]$

Let's just be clear that this change does not make Submariner support IPv6 based connections or routing, but to leverage and provide IPv4-based connectivity within clusters configured with dual-stack networking.

Do we plan to add CI gate for dual-stack K8S ?

Do we plan to add CI gate for validating IPV4 Submariner on dual-stack K8S ?

Its not on the immediate list of tasks, but we can plan for it as a low/medium priority task.

Ack,
I created https://github.com/submariner-io/submariner/issues/2308

@tpantelis can you take a re-look. Updated the PR.


Version 1.1 is not available

I can only find TGPassportAndroidSDK:1.0 on mavenCentral

Yeah, it doesn't show up on search.maven.org, even though I'm sure I pushed it and did everything exactly the way I did when I first released it. I'll try pushing it again.

Should be available now https://oss.sonatype.org/content/groups/public/org/telegram/passport/1.1/


added color and hover functionality

Added the Color and Hover Functionality of Social Media Icons in Teams Card

@Drontitan Can you check the animation and merge the PR


Coracle Social is nagging me to sign a kind 15 event

I don't know what a kind 15 event is, but the application is persistently requesting that I sign a kind 15 event, for which I'm denying

The details are a lot of event ids.

Kind 15 is for read status: https://github.com/nostr-protocol/nips/pull/933
The only way to make this kind of bookkeeping state interoperable is to publish it as nostr events, which triggers this kind of opaque dialog. That basically makes it impossible to micromanage permissions, so for now at least users will just have to trust Coracle to do the right thing.

ok. I'll refocus efforts on the NIP07 extension makers to add some support like Deny in addition to the cancel and confirm choices that Alby presents.  I haven't yet looked at how nos2x handles requests but its been a thorn to not be able to always ignore such requests when they come in.

I've temporarily disabled kind 15, so this one is closed until I figure out what I'm going to do about read receipts.


Create conda recipe for mrsimulator

We are having some issues with the binary distribution of the mrsimulator package for windows. An alternative is to look into writing a conda recipe for mrsimulator installation.

It's okay to have this issue open. It is not a priority at the moment, though. Conda has an advantage over pip in that it checks for dependency incompatibility. pip doesn't.


Problem defining a proxy option

This is reported by a customer @eng-khaled1 in azure-sdk-for-python repo:

Hello,
I am trying to configure a proxy option for the IOThubDeviceclient as suggested in the documentation:
client = IoTHubDeviceClient.create_from_connection_string(CONNECTION_STRING, proxy_options=( 'SOCKS5' , 'http://...*', '8883', None, None))
but I keep getting the error message
AttributeError: 'tuple' object has no attribute 'proxy_type'
i would be very thankfull if you could help me on it
im using azure-iothub-device-client 1.4.6 library

Hi there,
thanks a lot for your support
i installed azure-iot-device    2.8.0 but am  still getting the same error

@cartertinney,
yes, many many thanks.


Release mode not work

I am using the plugin to share photo and text, however, in the release version of the application sharing does not work.
When I'm using the debug version the sharing works perfectly.
Can someone help me?
void _shareImageAndText() async {
    try {
      var imageUrl = 'https://www.dropestore.com/img/07717000.jpg';
      var imageId = await ImageDownloader.downloadImage(imageUrl);
      var path = await ImageDownloader.findPath(imageId);
      var mimeType = await ImageDownloader.findMimeType(imageId);
      final ByteData bytes = await rootBundle.load(path);
      await WcFlutterShare.share(
          sharePopupTitle: 'share',
          subject: 'Title',
          text:
              'Body message',
          fileName: 'share.jpg',
          mimeType: mimeType,
          bytesOfFile: bytes.buffer.asUint8List());
    } catch (e) {
      print('error: $e');
    }
  }

Same I am using the same code and it's working fine when I am on debug mode. Can you please tell is this about any android permissions?
@aarajput

Have you tried adding internet permission in main manifest file?

Yes, I have added internet permission, read external permission and write external permission as well.

Is it throwing some exception?

@aarajput  No, when I use this in debug or on my emulator it works fine and pretty accurate but when I use this is my released apk/ built apk. It doesn't work at all

Have you tried example code?

Yes, it is not working with release mode

Strange. I will look into this.

@aarajput any updates, I need to use this in my app that is completed. Can you please tell me any solution?

@aarajput What I think is we need some other permissions as well her because in debug mode application have all the permissions but while release apk, there is a need for some more dependencies.

I am sorry, I didn't get time to look into this issue. Can you fork this repo and try to fix it and send me p.r?

@aarajput  I will try

There is an issue with your implementation. I have tested this on example.


LoganSquare has problems parsing ISO8601 dates?

{
"synced": "2016-12-14T21:32:50",
}
object model is:
@JsonObject
public class FileListResponse extends BaseResponse {
@JsonField(name="synced")
private Date lastSynced;
...
}

As the docs state, it has a default date formatter built in. If you want to use a different formatter, just add a type converter for it. The one that's built in is this format: yyyy-MM-dd'T'HH:mm:ss.SSSZ.
https://github.com/bluelinelabs/LoganSquare/blob/master/docs/TypeConverters.md

Thanks, Eric.
In case anyone else hits this w/ a server API that isn't correct...do this:
public class ISO8601DateConverter extends DateTypeConverter {

    private DateFormat mDateFormat;

    public ISO8601DateConverter() {
        mDateFormat = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss", Locale.getDefault());
    }

    public DateFormat getDateFormat() {
        return mDateFormat;
    }

}

And mark up your model's field like this:
@JsonField(name="synced", typeConverter=SSISO8601DateConverter.class)
private Date lastSynced;

I would look into making this into a type converter  https://github.com/realm/realm-java/blob/14e0e0695851b57b24c89691f4197bd93bb4e7a8/realm/realm-library/src/main/java/io/realm/internal/android/ISO8601Utils.java


Docs.MS CI Updates for Azure.Core.Amqp

CI Update for release of Azure.Core.Amqp

Docs Build status updates of commit a7e9a17:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

bundlepackages/azure-dotnet.csv
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Windows pathing issue on dev / build

Hi,
It looks as if there's an issue when running yarn dev or yarn build under windows
Seems to be some sort of pathing issue
√ Using Nuxt Typography v0.0.17                                                                                17:37:40
i Client built in 4227ms                                                                                17:37:51
i Building server...                                                                                17:37:51
√ Server built in 3795ms                                                                                17:37:55
√ Generated public .docs/.output/public                                                                                nitro 17:37:55
i Initializing prerenderer                                                                                nitro 17:37:55

 ERROR  Only URLs with a scheme in: file, data are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'

  at new NodeError (node:internal/errors:377:5)
  at throwIfUnsupportedURLScheme (node:internal/modules/esm/resolve:1076:11)
  at defaultResolve (node:internal/modules/esm/resolve:1156:3)
  at ESMLoader.resolve (node:internal/modules/esm/loader:605:30)
  at ESMLoader.getModuleJob (node:internal/modules/esm/loader:318:18)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:80:40)
  at link (node:internal/modules/esm/module_job:78:36)

I think it's related to the drive letter C: or D: in the path, although I've not been able to tie down exactly which bit of code is the problem.
It's the same issue mentioned here https://github.com/nuxt-themes/docus/issues/681 by KyleRicardo
someone else also mentioned it here - https://github.com/nuxt-themes/docus-starter/issues/7

The same. I've tried npm and pnpm.
Nuxt 3.0.0 with Nitro 1.0.0
√ Using Nuxt Typography v0.0.17

  > Local:    http://localhost:3000/
  > Network:  http://<IP_ADDRESS>:3000/

√ Nitro built in 1221 ms
i Vite client warmed up in 4596ms

 ERROR  [worker reload] [worker init] Only URLs with a scheme in: file, data are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'd:'

  at new NodeError (node:internal/errors:393:5)
  at throwIfUnsupportedURLScheme (node:internal/modules/esm/resolve:986:11)
  at defaultResolve (node:internal/modules/esm/resolve:1066:3)
  at nextResolve (node:internal/modules/esm/loader:161:28)
  at ESMLoader.resolve (node:internal/modules/esm/loader:831:30)
  at ESMLoader.getModuleJob (node:internal/modules/esm/loader:413:18)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:77:40)
  at link (node:internal/modules/esm/module_job:76:36)

I will raise my comment from #681 here again. We had a similar issue with Windows reading URL schemas differently then Mac or Windows. If it helps you can see how we approached the fix here:
https://github.com/sidebase/nuxt-session/pull/13

In my case it's windows 10 21H2
Tried from both a vanilla command prompt and a visual studio 2022 developer command prompt (same as a regular one but with additional tools installed)
Based on the above comments I'm guessing there's a import { resolve } from 'path' in a dependency some place that needs replacing with a
import { createResolver } from '@nuxt/kit'
const { resolve } = createResolver(import.meta.url)

similar to what's already done in the nuxt.config.ts
although the stack trace I'm getting isn't giving me any clues as to where it is other than it happening after the prerenderer

@grbd ; I've published this commit https://github.com/Tahul/pinceau/commit/5a8b4e92ef7ccbe27996765f3b8eaadb755b439d
I was using resolve from path and replaced it with pathe.
This might help on this issue, could you try reinstalling all dependencies with fresh lockfile?
Not sure this will help but it is first step.
Thanks 🙏

I tried doing a fresh clone to avoid any cache's or lock files being an issue
git clone https://github.com/nuxt-themes/docus
cd docus
yarn install

Then a yarn build / yarn dev (which tries to build inside .docs) but the result is still the same error sorry
also I double checked the version of pinceau was 0.6.26 the one you've just updated
Error is still
 ERROR  [worker reload] [worker init] Only URLs with a scheme in: file, data are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'

  at new NodeError (node:internal/errors:377:5)
  at throwIfUnsupportedURLScheme (node:internal/modules/esm/resolve:1076:11)
  at defaultResolve (node:internal/modules/esm/resolve:1156:3)
  at ESMLoader.resolve (node:internal/modules/esm/loader:605:30)
  at ESMLoader.getModuleJob (node:internal/modules/esm/loader:318:18)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:80:40)
  at link (node:internal/modules/esm/module_job:78:36)

I'm afraid we'll have to wait for me to have access to a Windows environment.
Thank you so much for the quick answer though.
I hope to fix this tomorrow, without being able to track which line throws this error I won't be able to fix this.

Nuxt/i18n had the same issue recently https://github.com/nuxt-community/i18n-module/issues/1518
They change the absolute path by a relative path with an assert type.
Hope it can help you

Just spotted a few places its showing up in the generated files under the .nuxt build directory
docus\.docs\.nuxt\app.config.mjs

import { defuFn } from 'C:/D/Temp/18/test3/docus/node_modules/defu/dist/defu.mjs'

const inlineConfig = {}

import cfg0 from "C:/D/Temp/18/test3/docus/.docs/app.config.ts"
import cfg1 from "C:/D/Temp/18/test3/docus/app.config.ts"
import cfg2 from<EMAIL_ADDRESS>
export default defuFn(cfg0, cfg1, cfg2, inlineConfig)

docus\.docs\.nuxt\dev\index.mjs
...
import _OxsnGi2RuD from 'C:/D/Temp/18/test3/docus/node_modules/pinceau/dist/nitro';
...

an entry within docus\.docs\.nuxt\dist\server\server.mjs.map
docus\.docs\.nuxt\prerender\chunks\nitro-prerenderer.mjs
...
import _OxsnGi2RuD from 'C:/D/Temp/18/test3/docus/node_modules/pinceau/dist/nitro';
...

Hey @grbd ;
That is immensely helpful, I suggest there is some issues in path resolving upstream.
@danielroe ; I've only identified this line in Nitro that seem not to use pathe:
https://github.com/unjs/nitro/blob/ffcbdbb2c65e43da1e985f952a25859ab298a140/src/runtime/entries/nitro-dev.ts#L4
Any idea on what could be the source of this? I would happily work on a PR to solve it.
Here is some code pointers that might help on digging:

Pinceau pushes Nitro plugin into nitro.plugins:

https://github.com/Tahul/pinceau/blob/e99c19bbca4ef4626c59d35bbd5bf652df6d2753/src/nuxt.ts#L44

Nuxt load and merges configs:

https://github.com/nuxt/framework/blob/71fed589c14eed3ebde5e1ccfd40e49651a3b0c6/packages/kit/src/loader/config.ts#L9

Nuxt writes the app.config template:

https://github.com/nuxt/framework/blob/71fed589c14eed3ebde5e1ccfd40e49651a3b0c6/packages/nuxt/src/core/templates.ts#L215

In the meantime I'm afraid I'll have to suggest running it in WSL.

After fix #686 at https://github.com/nuxt-themes/docus/blob/dev/app/module.ts#L5
only these broken pathes remain:
import 'C:/Users/.../node_modules/pinceau/dist/nitro'

So for the plugin line
https://github.com/Tahul/pinceau/blob/e99c19bbca4ef4626c59d35bbd5bf652df6d2753/src/nuxt.ts#L44
// I tried swapping
nuxt.options.nitro.plugins.push(modulePath.resolve('./nitro'))
// with an absolute path
nuxt.options.nitro.plugins.push('file:///C:/D/SourceCode/Forks/pinceau/dist/nitro')

However the path then ends up getting treated as a relative path and gets a bit mangled
 Cannot find module 'C:\D\Temp\18\.docs\server\file:\C:\D\SourceCode\Forks\pinceau\dist\nitro' imported from C:\D\Temp\18\test3\docus-orig\.docs\.nuxt\prerender\index.mjs
So the next question is whereabouts it's getting parsed ether in nuxt or nitro

Update, commenting that one line that adds the pinceau nitro plugin
https://github.com/Tahul/pinceau/blob/e99c19bbca4ef4626c59d35bbd5bf652df6d2753/src/nuxt.ts#L44
Allows us to get some semblence of an output we can preview
# switched to pnpm to get everything to build and link together during debugging
pnpm build
pnpm preview

But dev just hangs (probably because we're missing the plugin)
pnpm dev

I think the issue is someplace within nitro, I finally managed to get the depends to build and link together so more digging required.

@grbd ; are you saying the build passes with pnpm build and pnpm preview ?

@grbd ; are you saying the build passes with pnpm build and pnpm preview but dev is broken ?

Only if that line is commented out which removes the nitro plugin but that's not really a fix just an observation
I think I've discovered one of the problems in nitro so I've added a pull request for that
although we also need to add the .mjs extension to the path below within pinceau
# This
nuxt.options.nitro.plugins.push(modulePath.resolve('./nitro'))
# instead needs to be something like this
nuxt.options.nitro.plugins.push(modulePath.resolve('./nitro.mjs'))

Not sure if there's a better way but overall it gets us a little bit further along

I'm encountering the same issue using pnpm dev on Windows 11

i also encountered the same problem

Same here with win10

Hello @here !
Could you try regenerating your lockfiles to grab latest version of Pinceau, using Docus ^1.0.1?
I finally had access to a Windows dev environment and fixed it today!

@Tahul , yes, everything is fine, thank you!!👍

Same here 👍

Very cool works like a charm checked build, dev, preview
I'll close the issue down now

Working on win10. thank you!! 👏

So happy it fixed it for all of you, have fun documenting! 🙂


Why using $parent[0].lastChild

When trying to use a custom position for the loading bar, I must create two nested div in order for the bar to works (css are for demo only):

as this won't work:

Isn't it possible to append the bar to the currentSelector instead of parentSelector ?

I'm not following the exact question, but the title of your issue is related to this commit: c21a146.  Is this causing a problem?

Yes, I have the same problem. It is now not possible to use an empty element for the parent.
Also, it would be nice if one could use a query selector or an actual dom/jqlite.element object on parentSelector instead of $document.find which is restricted to tag names only.

I dont know -- that commit specifically was taken out of the Angular team's migration guide.  Are you using an older version of angular?

No, latest.
Can you maybe post an example where the loading bar is injected into an empty element? It doesn't show up for me at all if there is no element within the cfpLoadingBarProvider.parentSelector element.

Sorry for late reply, as omeid explained well enough, I'd like to use a generic selector instead of the parentSelector, so that we can inject the loader in any empty container.

I've pointed this out before. Just never got the time to write CoffeeScript tests.. https://github.com/chieffancypants/angular-loading-bar/pull/170#issuecomment-95126607

Added the PR: https://github.com/chieffancypants/angular-loading-bar/pull/218

See pull 272.


Remove extra BGR2RGB call in RetinaFaceWrapper

I suggest removing the extra BGR2RGB call.
While the RetinaFace model does expect RGB images, the corresponding conversion will be done in the RetinaFace preprocessing:
https://github.com/serengil/retinaface/blob/0c8d3ed8bd223c5753978a5e32549c25edff3316/retinaface/commons/preprocess.py#L45
The extra BGR2RGB call here will in fact make the final channel order wrong, because OpenCV does not know about the channel order, each call will simply swap the channels.
Thanks!

thank you for your contribution


added test to ensure benchmark is correct

The benchmark wasn't quite correct as some libs weren't even producing the same result.
I wrote a test compare the output of each benchmark to ts-belt.
I fixed the following issues

removed rambda flat test as their flatten is only a deep flatten
removed lodash/fp sort as their comparator api is different than the one being provided
fixed the ordering of zip for various libs

@mobily i think you might have accidentally re-added the lodash sortBy when merging https://github.com/mobily/ts-belt/pull/22/commits/08c3d123685a2907296aea2a111882e95437280f

oh, that's right, I'll remove it


FIX: HELM values file on master branch still points to the old "magnum" registry enpoint

Hi,
I was previously using an old release of the driver working properly on an EKS cluster.
Now I'm trying to upgrade it to the latest version (2.3.2) and I saw that the master branch points to the 'old' endpoint magnum on some references of the values.yaml file:
https://github.com/cvmfs-contrib/cvmfs-csi/blob/master/deployments/helm/cvmfs-csi/values.yaml#L72C1-L73C1
https://github.com/cvmfs-contrib/cvmfs-csi/blob/master/deployments/helm/cvmfs-csi/values.yaml#L80C52-L80C52
etc
This endpoint is not working anymore:
$ helm pull oci://registry.cern.ch/magnum/cvmfs-csi --version latest
Error: Unable to locate any tags in provided repository: oci://registry.cern.ch/magnum/cvmfs-csi

Should they need to point to the newer registry endpoint?
helm pull oci://registry.cern.ch/kubernetes/charts/cvmfs-csi --version 2.3.2
Pulled: registry.cern.ch/kubernetes/charts/cvmfs-csi:2.3.2
Digest: sha256:f016d4a1488705d8cddd67f7fb48caffbe564e61f1d0544cf7a33c4285a591cf

Created a PR on: #130

Done in https://github.com/cvmfs-contrib/cvmfs-csi/pull/130. Thanks for the work @jtorrex, closing.


BUG in computeGaussianPerplexity when K > N - 1

In line tsne.cpp#L37 vectors indices and distances will have < K + 1 elements if K > N - 1.
This will cause erroneous reads out of the bounds of the vector in tsne.cpp#L388 and other loops.
There should be a check that K <= N -1.

Hi Artsiom, thanks for reporting. Can you please submit a pull request?


[actions] automatically answer missing windows SDK issues

This kind of issue is reported over and over again and the answer is always the same: Install the windows SDK. So lets automate this.

I think this must be solved one step earlier: Clearly indicate in vcpkg tool's reporting that the SDK is missing, instead of suggesting to open an issue.

@dg0yt But I guess vcpkg_cmake_configure could detect that in the logs and suggest to install the windows SDK

@Adela0814 Can you add the  info:reviewed Pull Request changes follow basic guidelines label back?

No, because I'm merging it. Thanks!


Keeps Proposal

@thayeylolu what is this PR doing ? The commits are connected with script but the title is about proposal. Does it only create the md file ?

@thayeylolu what is this PR doing ? The commits are connected with script but the title is about proposal. Does it only create the md file ?

Yeah I asked @thayeylolu to create a proposal md just to store our readme from the previous milestone. I think she made other other changes to the cleaning and preprocessing scripts as well. @thayeylolu please confirm this and let us know about the other changes you made, thanks!

Yes, i created the the proposal as you mentioned. i believe i dont have changes to add

but we can ignore it if we dont need it

but we can ignore it if we dont need it

We need a proposal, I just wanted to know what where the commits about. I guess there was some confusion with commits history, but if this PR is creating md file then it is good to go. @thayeylolu @Arushi282

@thayeylolu After discussion we are cancelling this PR


java.lang.IllegalAccessException: Should implement the OnPopupTextUpdate or OnPopupViewUpdate interface

Why does it raise an exception?
Just not showing the popup when it's not implemented makes much more sense

@tom5079 let's make a point to reference issues in commit messages like "Fixes #27", this way, whenever the commit is merged to master, github closes the issue with the reference commit.


Missing atrribute in schema

Hi,
I have problem with parsing attributes and I do not know if I missing something or if this is a bug.
I will try to demonstrate it on following minimal example. This is the minimal blueprint file:
FORMAT: 1A

# Minimal example API

Example of minimal API

# Group Authentication

## Authentication [/et/v1/{id}/auth]
Authentication...

+ Parameters
    + id: `42` (number) - example id

### Initialize Authentication [GET]

+ Response 200 (application/json)
    + Attributes (AuthenticationModel)

# Data Structures

## BaseModel (object)
+ _links (string)

## AuthenticationModel (BaseModel)
+ example (boolean)

And this is corresponding json output, generated using apib2json.
{
  "[GET]/et/v1/{id}/auth": [
    {
      "meta": {
        "type": "response",
        "title": null,
        "group": "Authentication",
        "statusCode": "200"
      },
      "schema": {
        "$schema": "http://json-schema.org/draft-04/schema#",
        "type": "object",
        "properties": {
          "_links": {
            "type": "string"
          },
          "example": {
            "type": "boolean"
          }
        }
      }
    }
  ]
}

Unfortunately, I cannot find any information about id parameter in generated JSON file. Am I missing something or is it a bug?

Hello @starek4, you defined id as GET parameter so this parameter cannot be in JSON Schema (which is primally for request / response body).
This is not a behaviour of apib2json but Apiary's protagonist. If you need to validate this parameter, use for example POST method.


Add optional -d argument to output dotfile.

In case you want to fine tune the output before passing it to graphviz.

It would be perfect if you encapsulated the Io in a function.
Just as a curiosity, how do you customise the output?
Le 14 avr. 2016 21:02, "Dan Root"<EMAIL_ADDRESS>a écrit :

In case you want to fine tune the output before passing it to graphviz.
You can view, comment on, or merge this pull request online at:
https://github.com/Alexis-benoist/draw-compose/pull/1
Commit Summary

Add optional -d argument to output dotfile.

File Changes

M draw_compose/draw_compose.py
https://github.com/Alexis-benoist/draw-compose/pull/1/files#diff-0
(4)

Patch Links:

https://github.com/Alexis-benoist/draw-compose/pull/1.patch
https://github.com/Alexis-benoist/draw-compose/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/Alexis-benoist/draw-compose/pull/1

Just made the change suggested.
Mostly I was looking to change the layout by using some subgraph ranking.   Pin the data stores to max rank, while pinning the containers that interact directly with the outside world to the min rank.   The document was for some internal team system diagrams, so I also ended up fiddling with some of the edge weights to force various containers closer to logically similar containers.

Cool!
Do you mind sharing with me the graphviz code you used?
Le 16 avr. 2016 19:28, "Dan Root"<EMAIL_ADDRESS>a écrit :

Just made the change suggested.
Mostly I was looking to change the layout by using some subgraph ranking.
Pin the data stores to max rank, while pinning the containers that interact
directly with the outside world to the min rank. The document was for some
internal team system diagrams, so I also ended up fiddling with some of the
edge weights to force various containers closer to logically similar
containers.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/Alexis-benoist/draw-compose/pull/1#issuecomment-210861579

Here you go, safely anonymized.
The additions are just a couple of ranks and a couple of weights to make sure that appropriate clusters end up together.
digraph {
    graph [rankdir=LR];
    node [label="\N",
        shape=box,
        style=filled,
        color=".7 .3 1.0"
    ];
    edge [color=gray50,
	arrowhead=normal,
	minlen=2
    ];
"a2";
"v5";
"v3";
"v4";
"web3";
"dl1";
"ac2";
"ac3";
"web1";
"v2";
"db1";
"apip";
"api2";
"api1";
"v1";
"ac1";
"db2";
"nd1";
"web2";
"a3";
"db4";
"n1";
"a1";
"a2" -> "a1"
"a2" -> "ac3"
"a2" -> "n2"
"v5" -> "v1"
"v5" -> "db1"
"v5" -> "ac1"
"v5" -> "v3" [weight=50];
"v3" -> "db1"
"v3" -> "ac1"
"v3" -> "ac2"
"v3" -> "v6"
"v4" -> "db1"
"v4" -> "ac1"
"v4" -> "v1" [weight=100];
"web3" -> "db1"
"web3" -> "db3"
"web3" -> "ac2"
"web3" -> "q1"
"dl1" -> "db4"
"dl1" -> "wd1"
"ac1" -> "ac2"
"ac2" -> "db1"
"ac2" -> "db2"
"ac2" -> "db3"
"ac2" -> "wd1"
"ac3" -> "nd1"
"ac3" -> "n1"
"ac3" -> "a3"
"ac3" -> "db1"
"ac3" -> "db5"
"ac3" -> "f1"
"web1" -> "api1"
"web1" -> "db1"
"web1" -> "ac2"
"web1" -> "db3"
"web1" -> "q1"
"v2" -> "v4" [weight=100];
"v2" -> "db1"
"v2" -> "ac2"
"api2" -> "db1"
"api2" -> "a2"
"api2" -> "ac3"
"api1" -> "db1"
"api1" -> "db3"
"apip" -> "api1" [weight=75];
"apip" -> "api2" [weight=75];
"v1" -> "v4" [weight=100];
"v1" -> "ac1"
"v1" -> "db1"
"ac1" -> "db1"
"ac1" -> "db4"
"ac1" -> "db3"
"ac1" -> "n1"
"web2" -> "db1"
"web2" -> "q1"
"web2" -> "db3"
"n2" -> "nd1" [weight=75];
"n1" -> "nd1" [weight=75];
"a1" -> "a2"

{ rank=min; a1; v1; v2; v3; dl1; web1; }
{ rank=same; db2; db3; db4; }
{ rank=max; db1; db5; }
}


Name container by default with Image alias when starting container

If an image has an <alias>, the container created during docker:start should be named with the alias. If already a container with name exists, then a suffix should be added until the name is unique. E.g. if the alias db is alread taken,  try db_1, db_2, .... until a free name is found.
See #35 for a discussion about this.

i have a basic implementation of this working but it requires the open pull requests for #108 to be merged first as it's built on top of that.
this is what the config looks like
<image>
  <name>busybox:latest</name>
  <alias>issue-48-named</alias>
  <namingScheme>alias</namingScheme>
  <run>
    <command>ping google.com</command>
  </run>
</image>

namingScheme is an enum that supports the following two values: none and alias w/ the default being none.
this resolves the use case i need to satisfy right now and the flexibility is there for more naming schemes to be added at a later date if the demand is there but for now i don't see the need.
open to suggestions if you don't like namingScheme as the element name and/or none and alias as the enum values.

this is ready to go pending the complete of the outstanding pull requests and any changes i need to sync w/ as a result.
i moved the namingScheme element into the run configuration where it makes more sense:
<image>
  <name>busybox:latest</name>
  <alias>issue-48-named</alias>
  <run>
    <command>ping google.com</command>
    <namingScheme>alias</namingScheme>
  </run>
</image>

it would be super great if 0.11.2 could be released once all the PRs (including this one when it goes up) get merged in.

I'm halfway through the PRs and try my very best to release this weekend ;)

cool - i just didn't want to overwhelm you anymore then i did w/ changes stacked on changed. as soon as #110 goes in i'll sync on my end and create this PR.

Fixed in 0.11.3


Dockerfile: copy pkg source folder

Should fix latest failure on main container build: https://github.com/opendatahub-io/model-registry/actions/runs/6860664179/job/18654888839
#20 20.62 go: finding module for package github.com/opendatahub-io/model-registry/pkg/api
#20 20.62 go: finding module for package github.com/opendatahub-io/model-registry/pkg/core
#20 24.28 github.com/opendatahub-io/model-registry/cmd imports
#20 24.28 	github.com/opendatahub-io/model-registry/pkg/core: no matching versions for query "latest"
#20 24.28 github.com/opendatahub-io/model-registry/internal/apiutils imports
#20 24.28 	github.com/opendatahub-io/model-registry/pkg/api: no matching versions for query "latest"
#20 24.28 tidy failed: go mod tidy failed: exit status 1
#20 24.29 make: *** [Makefile:71: internal/model/graph/models_gen.go] Error 1
#20 ERROR: process "/bin/sh -c CGO_ENABLED=1 GOOS=linux GOARCH=amd64 make clean model-registry" did not complete successfully: exit code: 2

Description
Copy /pkg source folder in the Dockerfile
How Has This Been Tested?

Merge criteria:

[x] The commits and have meaningful messages; the author will squash them after approval or will ask to merge with squash.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

Change incorporated in https://github.com/opendatahub-io/model-registry/pull/139, closing this.


Sharing color schemes

I love the fact that we have the ability to experiment with color schemes in the theme now, but I don't see a way to share custom color schemes or find ones from the community. It would be great to have a place to find, try and share more custom color schemes.

See official documentation on this topic:
https://minimal.guide/Plugins/Style+Settings
Style Settings offers the option to share color schemes by using the Import and Export buttons in the top right corner of the Style Settings pane.
Note that when using the Export functionality, all of your saved Style Settings preferences across all themes you've modified will be exported. You may need to edit the output to only include the styles that you want to share.
You can find and share Minimal color schemes in the Obsidian forum thread or in the the Minimal channel on the official Obsidian Discord.

Got it. Thanks!


Error with required type: password in rancher compose

Rancher Version:
v1.2.0-pre2
Docker Version:
1.12.1
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 16.04
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Create configuration where you have type:password in rancher.compose.yml. Example:
.catalog:
  version: 1.0.0
  questions:
    - variable: github_client_secret
      label: GitHub client secret
      required: true
      type: password
    - variable: drone_secret
      label: secret used by Drone agent to communicate with Server
      required: true
      type: password

When adding that value via the interface, the added value is not accepted unless the button 'Generate' is used. Example:

Curiously the other type:password (the one right below in the example picture) works fine.
Results:
Unable to launch stack unless 'Generate' button is used to fill in the field.
Expected:
All type:password fields should allow being introduced.
Update
Digging a little deeper, I've found that any required password fields in the first row are not accepted. Example rancher-compose.yml:
.catalog:
  version: 1.0.0
  questions:
    - variable: secret_var_1
      label: Secret Var 1
      required: true
      type: password
    - variable: secret_var_2
      label: Secret Var 2
      required: true
      type: password
    - variable: secret_var_3
      label: Secret Var 3
      required: true
      type: password

Check pictures below.

As you can see in the first picture, when all fields are empty, Rancher behaves as expected. In the second picture we can see that, although, all fields are filled in, only the bottom one is accepted.

This appears to have been because they all get the same id="password" in the HTML, which makes only the last password field work (not one on the first row, etc)
Anyway this was fixed right after 1.2.0-pre3 in https://github.com/rancher/ui/commit/52506581ba9f80e99d3cea7e3a7dc7156492858a#diff-b8a249cd297ed3c367887a8adc9059e9L1

Version - master 11/09
Verified fixed


Replace spaces with %20 in path names

If there are strings in the path name, the sbt-script argument will be
interpreted as multiple arguments. The thin client also needs to be
updated to handle %20 in path names.

Today we can do
sbt "eval \"hello world\""

Is there something fundamentally different about passing whitespace into sbtn?

No, but if you sbt.bat --sbt-script=C:\Program Files\sbt\bin\sbt.bat, the arbuments will be ('--sbt-script=C:\Program', 'Files\sbt\bin\sbt.bat'). If we quote it, then we'd have to strip the quotes. This change will only affect the --sbt-script parameter.

I am pretty sure that if you did sbt --client "--sbt-script=C:\Program Files\sbt\bin\sbt.bat" in the windows command prompt, it would work fine with or without this change. The problem is that the SBT_ARGS variable is a giant string, not an array so it can't distinguish spaces and quotes don't automatically get stripped like they would if you are running from the command line.

Here is a simple bash script that demonstrates the problem. I created an alternate sbt script that has an absolute path with a space in it:
#!/bin/bash

arg="\"--sbt-script=/Users/ethanatkins/work/scratch/scala-compile/bin foo/sbt\" compile"
sbtn $arg

If you run this, you get the following output:
[info] entering *experimental* thin client - BEEP WHIRR
[info] terminate the server with `shutdown`
> "--sbt-script=/Users/ethanatkins/work/scratch/scala-compile/bin foo/sbt" compile
[error] Expected letter
[error] Expected symbol
...
[error] "--sbt-script=/Users/ethanatkins/work/scratch/scala-compile/bin foo/sbt" compile
[error] ^
[info] "--sbt-script=/Users/ethanatkins/work/scratch/scala-compile/bin foo/sbt" compile completed
[error] Total time: 0 s, completed Oct 25, 2020, 7:56:33 PM

If instead the script were
#!/bin/bash

arg="--sbt-script=/Users/ethanatkins/work/scratch/scala-compile/bin foo/sbt compile"
sbtn $arg

you get:
/Users/ethanatkins/work/scratch/scala-compile/bin
[info] entering *experimental* thin client - BEEP WHIRR
[info] terminate the server with `shutdown`
> foo/sbt compile
[error] Expected ID character
[error] Not a valid command: foo (similar: loop, boot, reboot)
[error] Expected project ID
[error] Expected configuration
[error] Expected ':'
[error] Expected key
[error] Not a valid key: foo (similar: fork, doc, onLoad)
[error] foo/sbt compile
[error]    ^
[info] foo/sbt compile completed
[error] Total time: 0 s, completed Oct 25, 2020, 7:57:21 PM

The problem is that neither bash nor batch scripts give you the ability to actually pass in the command arguments as arrays which makes it really difficult to parse arguments that have spaces in them. Fish shell is different and does actually allow you to use an array of arguments, ala java process builder, so that spaces aren't syntactically significant in this way.

Your eval example works fine with sbtn I think because the eval command handles quotation marks in its parser.

The problem is that neither bash nor batch scripts give you the ability to actually pass in the command arguments as arrays which makes it really difficult to parse arguments that have spaces in them.

ok. That makes sense.


Add logging around failure to write report

Is your feature request related to a problem? Please describe.
Yes, it is related to a problem.
When gitleaks fails to write the report file, apparently it fails silently. See discussion on this issue: https://github.com/zricethezav/gitleaks/issues/988
Describe the solution you'd like
There are two solutions:

When gitleaks fails to write the report file, should log a useful warning or error message.
Ideally, gitleaks could check file/directory permissions before beginning the scan to ensure that it will be able to write the report at the end. If it doesn't have permission to write to that location, fail immediately with useful error.

Describe alternatives you've considered
None
Additional context
See the discussion on this issue: https://github.com/zricethezav/gitleaks/issues/988
cc @zricethezav

Update:
I actually do see logging when the report fails to write.
Command:
./gitleaks detect --source ./ --no-git --report-path /Users/andrew/root-owned/file.json

Result:
    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

9:12AM INF scan completed in 94.5ms
9:12AM WRN leaks found: 46
9:12AM FTL could not write error="open /Users/andrew/root-owned/file.json: permission denied

Need to investigate further to understand why the failure in #988 wasn't being logged...

When I run using docker, the FTL log is not printed to console:
docker run -v /Users/andrew/projects/gitleaks-projects/zricethezav/gitleaks:/home/gitleaks/src zricethezav/gitleaks detect --source /home/gitleaks/src --report-path /fake/directory/111.json

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

4:28PM INF scan completed in 5.2s
4:28PM WRN leaks found: 18

However, when I build the docker image from source, and then run that image, it does print the FTL log to console:
docker build . -t gitleaks-local

docker run -v /Users/andrew/projects/gitleaks-projects/zricethezav/gitleaks:/home/gitleaks/src gitleaks-local detect --source /home/gitleaks/src --report-path /fake/directory/111.json

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

4:32PM INF 714 commits scanned.
4:32PM INF scan completed in 5.24s
4:32PM WRN leaks found: 18
4:32PM FTL could not write error="open /fake/directory/111.json: no such file or directory"

So why is the behavior of the locally built docker image different than the released docker image? 🤔
@zricethezav Any thoughts?

Todo

[ ] Double check that with the released docker image, the report file is not being written
[ ] Find another fatal failure to trigger. See if the released docker image logs that FTL log.

So why is the behavior of the locally built docker image different than the released docker image? 🤔
@zricethezav Any thoughts?

No idea


too many open files?

I know this sort of stuff has been discussed extensively in the past, but I feel that I don't have enough containers to have run into this issue.  I have four days uptime on a rancherOS node, and yet with just a handful of containers running I'm hitting the too many open files wall.  I can certainly fix things from here and get it working by doing some cleanup, but I feel very much that I am just treating the symptom of a larger issue.  I suspect the root cause is something with overlay itself piling up openfiles.
rancher@rancher:~$ docker run --name $OVPN_DATA -v /etc/openvpn busybox
FATA[0000] Error response from daemon: open /var/lib/docker/repositories-overlay: too many open files
rancher@rancher:~$ docker ps -a
CONTAINER ID        IMAGE                             COMMAND                CREATED             STATUS                       PORTS                                          NAMES
b50eb510e8ae        rancher/agent-instance:v0.3.1     "/etc/init.d/agent-i   3 days ago          Up 3 days                                                                   a30d13a2-b6c0-4918-be81-4ddad1f91ead
216acc9c7ed9        nginx:latest                      "nginx -g 'daemon of   3 days ago          Up 3 days                    80/tcp, 443/tcp                                9c042341-ee20-48f4-ba1c-fff2103cc0fe
d054276ff9da        mysql:latest                      "/entrypoint.sh mysq   3 days ago          Up 3 days                    3306/tcp                                       00b0bb7e-315b-4ba4-902a-d5dfe7958e1d
1bdd4a1be033        rancher/agent:v0.7.9              "/run.sh run"          3 days ago          Up 3 days                                                                   rancher-agent
8257cff9bb54        joshuacox/docker-icinga2:latest   "/entrypoint.sh /usr   4 days ago          Up 4 days                    80/tcp                                         54a086b4-979b-4bcb-9053-938323279085
c913c04cc127        rancher/agent-instance:v0.3.1     "/etc/init.d/agent-i   4 days ago          Up 4 days                    <IP_ADDRESS>:500->500/udp, <IP_ADDRESS>:4500->4500/udp   74525b44-5a30-40dc-8f0d-37a9418de210
e9c18cc8e015        rancher/agent:v0.7.9              "/run.sh state"        4 days ago          Exited (0) 4 days ago                                                       rancher-agent-state
bb5ca9d03bc1        rancher/agent:v0.7.9              "/run.sh http://ranc   4 days ago          Restarting (-1) 2 days ago

@joshuacox can you please add your comments to https://github.com/rancherio/os/issues/359

This isn't just a RancherOS issue as I've had the same issue reproduced on RHEL 6.

@deniseschannon can you confirm what storage driver you are using?  Notice I'm using overlay here, but I also have hosts using AUFS and BTRFS this problem is singular with overlay,  though I have separate problems with the other storages as well.

When this error occurs can you please share the output of "lsof -p ".  I believe this is a bug in Docker 1.6

I hit the same issue. here is the output of lsof -p :
31536	/bin/bash	/dev/pts/0
31536	/bin/bash	/dev/pts/0
31536	/bin/bash	/dev/pts/0
31536	/bin/bash	/dev/pts/0

Is there a workaround ? I can't event stop container or list images.

use ps to find and then kill the process of the rancher-agent

FYI, I've just set ulimit -n to 1000000 and  /proc/sys/fs/file-max to<PHONE_NUMBER> in v0.4.0-dev. This is coming to v0.4.0-rc2 to be cut in a few minutes.

@imikushin, how can I change these settings in an existing, running RancherOS v0.3.3 server? I need to up the open file limit for a database container but ulimit -n unlimited gives me ulimit: open files: cannot modify limit: Operation not permitted and modifying /etc/security/limit.conf doesn't seem to have any affect.

Thanks @EliSnow!
So, you can add --default-ulimit, nofile=1000000:1000000 to docker args via cloud-config. On RancherOS v0.3.3 this should look like:
#cloud-config
rancher:
  user_docker:
    args: [docker, -d, -s, overlay, -G, docker, -H, 'unix:///var/run/docker.sock', --default-ulimit, nofile=1000000:1000000]

Or, using ros config:
# ros config set rancher.user_docker.args [docker, -d, -s, overlay, -G, docker, -H, 'unix:///var/run/docker.sock', --default-ulimit, nofile=1000000:1000000]

I tried increasing my docker container ulimits using the ros config listed above and was told Incorrect usage. Should I change from what is listed by imikishin?

@eyecreate which version of rancherOS are you using?
In RancherOS, v0.3.3, it should probably be
ros config set user_docker.args [docker, -d, -s, overlay, -G, docker, -H, 'unix:///var/run/docker.sock', --default-ulimit, nofile=1000000:1000000]

In v0.4.0, we require the rancher prefix as we are allowing cloud configs (includes write_files, ssh_authorized_keys) to be saved versus just the rancher configurations.

I had the same issue I guess as @eyecreate, every time I run the mentioned command I get
$ sudo ros config set user_docker.args [docker, -d, -s, overlay, -G, docker, -H, 'unix:///var/run/docker.sock', --default-ulimit, nofile=1000000:1000000]
Incorrect Usage.

NAME:
   set - set a value

USAGE:
   command set [arguments...]

So it turns out that it need quotation to work out
$ sudo ros config set user_docker.args '[docker, -d, -s, overlay, -G, docker, -H, 'unix:///var/run/docker.sock', --default-ulimit, nofile=1000000:1000000]'

Closing issue as we don't support RHEL 6 and we will track it as part of RancherOS.


Uniform the setting of init code size limit

Signed-off-by: Fabio Di Fabio<EMAIL_ADDRESS>

PR description
Fixed Issue(s)

Documentation

[ ] I thought about documentation and added the doc-change-required label to this PR if
updates are required.

Changelog

[ ] I thought about the changelog and included a changelog update if required.

Closing since init code limit is different from code size see https://github.com/hyperledger/besu/pull/4911#discussion_r1070406464


Update syllabus.md

Added my name and my paper to 6b

Unfortunately, I just accepted a PR that I believe changed the paper name. Mind updating?

I just checked and the name for 6b is correct

https://github.com/gwu-iot/collaboration/blob/master/papers/iom11medical_security.pdf
Go to that address. Does it resolve for you?

okay I think i fixed it

Thanks for all of this!